Skip to content

fix(autofix): Only call the repo check if it's github#80847

Merged
roaga merged 1 commit into
masterfrom
jenn/autofix/repo-check-fix
Nov 18, 2024
Merged

fix(autofix): Only call the repo check if it's github#80847
roaga merged 1 commit into
masterfrom
jenn/autofix/repo-check-fix

Conversation

@jennmueng

Copy link
Copy Markdown
Contributor

Don't call the repo access check endpoint if it's not a github provider.

@jennmueng jennmueng requested a review from a team as a code owner November 15, 2024 19:43
@jennmueng jennmueng requested a review from roaga November 15, 2024 19:43
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 15, 2024

@roaga roaga left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ty

@codecov

codecov Bot commented Nov 15, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../sentry/api/endpoints/group_autofix_setup_check.py 33.33% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #80847   +/-   ##
=======================================
  Coverage   78.41%   78.41%           
=======================================
  Files        7209     7209           
  Lines      319622   319625    +3     
  Branches    44006    44007    +1     
=======================================
+ Hits       250642   250647    +5     
  Misses      62594    62594           
+ Partials     6386     6384    -2     

@roaga roaga merged commit bceabb7 into master Nov 18, 2024
@roaga roaga deleted the jenn/autofix/repo-check-fix branch November 18, 2024 16:13
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 4, 2024
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