Skip to content

fix(preprod): Post NEUTRAL status check when all artifacts are skipped#111811

Open
mtopo27 wants to merge 4 commits intomasterfrom
mtopo27/fix-size-status-check-for-all-skipped-condition
Open

fix(preprod): Post NEUTRAL status check when all artifacts are skipped#111811
mtopo27 wants to merge 4 commits intomasterfrom
mtopo27/fix-size-status-check-for-all-skipped-condition

Conversation

@mtopo27
Copy link
Copy Markdown
Contributor

@mtopo27 mtopo27 commented Mar 30, 2026

When all artifacts for a commit are filtered/skipped by project settings, the
status check task returned early without updating any previously posted
IN_PROGRESS check on GitHub. This left checks stuck as in_progress forever.

Confirmed in production: artifact uploads would initially post an IN_PROGRESS
check (before the filter decision), then a subsequent task invocation would see
all artifacts as SKIPPED and hit the early return at tasks.py:256-261 — never
updating the check to a terminal state.

Now the all_skipped branch posts a NEUTRAL completed check with a "Size
analysis skipped" message and a link to configure status check rules, matching
the existing pattern for the no-quota case.

When all artifacts for a commit are filtered/skipped by project
settings, the status check task returned early without updating any
previously posted IN_PROGRESS check on GitHub. This left checks stuck
as in_progress forever.

Now posts a NEUTRAL completed check with a "Size analysis skipped"
message and a link to configure status check rules, matching the
existing pattern for the no-quota case.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 30, 2026
@mtopo27 mtopo27 marked this pull request as ready for review March 30, 2026 20:22
@mtopo27 mtopo27 requested a review from a team as a code owner March 30, 2026 20:22
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