Skip to content

chore(integrations): Move repository syncing code to generic place.#112431

Merged
wedamija merged 1 commit intomasterfrom
danf/move-sync-repos-task
Apr 9, 2026
Merged

chore(integrations): Move repository syncing code to generic place.#112431
wedamija merged 1 commit intomasterfrom
danf/move-sync-repos-task

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented Apr 7, 2026

This moves this code into a more generic location, but doesn't change it at all. Will follow up with the changes to make this work for all SCMs.

@wedamija wedamija requested a review from a team April 7, 2026 23:24
@wedamija wedamija requested review from a team as code owners April 7, 2026 23:24
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 7, 2026
@wedamija wedamija force-pushed the danf/move-sync-repos-task branch 3 times, most recently from cb39064 to 987d043 Compare April 7, 2026 23:29
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 987d043. Configure here.

Base automatically changed from danf/repo-external-id-refactor to master April 8, 2026 20:26
This moves this code into a more generic location, but doesn't change it at all. Will follow up with the changes to make this work for all SCMs.
"github-repo-sync-beat": {
"task": "integrations.control:sentry.integrations.github.tasks.sync_repos.github_repo_sync_beat",
"scm-repo-sync-beat": {
"task": "integrations.control:sentry.integrations.source_code_management.sync_repos.scm_repo_sync_beat",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a generic task or a GitHub task?

Copy link
Copy Markdown
Member Author

@wedamija wedamija Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will become generic in a follow up pr, just for review purposes it's easier to move then modify it

@wedamija wedamija merged commit 14deaf5 into master Apr 9, 2026
77 checks passed
@wedamija wedamija deleted the danf/move-sync-repos-task branch April 9, 2026 17:54
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…112431)

This moves this code into a more generic location, but doesn't change it
at all. Will follow up with the changes to make this work for all SCMs.

<!-- Describe your PR here. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants