Skip to content

chore(repositories): Abstract broken integration handling into base class#114464

Merged
wedamija merged 5 commits intomasterfrom
danf/sync-repos-broken-integration-abstraction
Apr 30, 2026
Merged

chore(repositories): Abstract broken integration handling into base class#114464
wedamija merged 5 commits intomasterfrom
danf/sync-repos-broken-integration-abstraction

Conversation

@wedamija
Copy link
Copy Markdown
Member

It's getting increasingly messy handling all of these errors. Consolidating these into the RepositoryIntegration so that we can override it per provider

@wedamija wedamija requested a review from a team April 30, 2026 18:17
@wedamija wedamija requested review from a team as code owners April 30, 2026 18:17
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 30, 2026
Comment thread src/sentry/integrations/source_code_management/repository.py Outdated
…lass

It's getting increasingly messy handling all of these errors. Consolidating these into the `RepositoryIntegration` so that we can override it per provider
@wedamija wedamija force-pushed the danf/sync-repos-broken-integration-abstraction branch from 1f350c7 to f1820b6 Compare April 30, 2026 22:05
@wedamija wedamija enabled auto-merge (squash) April 30, 2026 22:33
Comment thread src/sentry/integrations/source_code_management/repository.py Outdated
@wedamija wedamija disabled auto-merge April 30, 2026 22:39
@wedamija wedamija enabled auto-merge (squash) April 30, 2026 22:43
@wedamija wedamija merged commit ac6de89 into master Apr 30, 2026
60 checks passed
@wedamija wedamija deleted the danf/sync-repos-broken-integration-abstraction branch April 30, 2026 22:52
cleptric pushed a commit that referenced this pull request May 5, 2026
…lass (#114464)

It's getting increasingly messy handling all of these errors.
Consolidating these into the `RepositoryIntegration` so that we can
override it per provider

<!-- 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