Skip to content

fix(typing): Type src/sentry/integrations/source_code_management#102704

Merged
thetruecpaul merged 1 commit into
masterfrom
cpaul/110425/typetostart
Nov 5, 2025
Merged

fix(typing): Type src/sentry/integrations/source_code_management#102704
thetruecpaul merged 1 commit into
masterfrom
cpaul/110425/typetostart

Conversation

@thetruecpaul

Copy link
Copy Markdown
Contributor

Pretty simple addition of types. get_repository_choices has a weird command flow that requires an assert for mypy to recognize that a path is unreachable, but I feel confident in the correctness there.

Pretty simple addition of types. `get_repository_choices` has a weird command flow that requires an assert for mypy to recognize that a path is unreachable, but I feel confident in the correctness there.
@thetruecpaul thetruecpaul requested a review from a team November 4, 2025 18:59
@thetruecpaul thetruecpaul requested review from a team as code owners November 4, 2025 18:59
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 4, 2025
@codecov

codecov Bot commented Nov 4, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ntry/integrations/source_code_management/issues.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #102704      +/-   ##
===========================================
+ Coverage   80.91%    80.93%   +0.02%     
===========================================
  Files        8929      8929              
  Lines      391242    391284      +42     
  Branches    24866     24866              
===========================================
+ Hits       316574    316688     +114     
+ Misses      74300     74228      -72     
  Partials      368       368              

@thetruecpaul thetruecpaul merged commit 68499cf into master Nov 5, 2025
69 checks passed
@thetruecpaul thetruecpaul deleted the cpaul/110425/typetostart branch November 5, 2025 18:59
priscilawebdev pushed a commit that referenced this pull request Nov 6, 2025
…2704)

Pretty simple addition of types. `get_repository_choices` has a weird
command flow that requires an assert for mypy to recognize that a path
is unreachable, but I feel confident in the correctness there.
Jesse-Box pushed a commit that referenced this pull request Nov 12, 2025
…2704)

Pretty simple addition of types. `get_repository_choices` has a weird
command flow that requires an assert for mypy to recognize that a path
is unreachable, but I feel confident in the correctness there.
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
…2704)

Pretty simple addition of types. `get_repository_choices` has a weird
command flow that requires an assert for mypy to recognize that a path
is unreachable, but I feel confident in the correctness there.
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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