Skip to content

Conversation

@trevor-e
Copy link
Member

Records in the extras field whether the status check succeeded or not, which we can later display in the UI.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 26, 2025
@trevor-e trevor-e marked this pull request as ready for review November 26, 2025 03:25
@trevor-e trevor-e requested a review from a team as a code owner November 26, 2025 03:25

posted_status_checks = extras.get("posted_status_checks", {})

check_result: dict[str, Any] = {"success": success}
Copy link
Contributor

Choose a reason for hiding this comment

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

minor but you could move these lines out of the transaction

@codecov
Copy link

codecov bot commented Nov 26, 2025

Codecov Report

❌ Patch coverage is 54.05405% with 17 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/preprod/vcs/status_checks/size/tasks.py 54.05% 17 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104039      +/-   ##
===========================================
+ Coverage   77.47%    80.60%   +3.13%     
===========================================
  Files        9316      9318       +2     
  Lines      397607    397582      -25     
  Branches    25398     25379      -19     
===========================================
+ Hits       308032    320480   +12448     
+ Misses      88907     76650   -12257     
+ Partials      668       452     -216     

@trevor-e trevor-e merged commit 2c48c43 into master Nov 26, 2025
67 checks passed
@trevor-e trevor-e deleted the telkins/status-check-track branch November 26, 2025 18:15
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.

3 participants