Repository navigation
fix(ci): name the failed Fix Exporter verify check and re-read a lagging artifact listing - #905
Conversation
…ing artifact listing Each verify assertion now prints the check, the expected value and the actual value before failing, instead of ending with a bare exit code. The run's artifact listing is re-read (up to six reads, five seconds apart) while it shows fewer artifacts than expected; the count must still match exactly, so a missing or extra artifact fails. A new section of the workflow-fix test drives the verify phase against a stubbed listing. Part of #904 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Code coverage overviewLanguages: Go, C# Go / code-coverage/test
C# / code-coverage/dotnet
Go / code-coverage/go
ℹ️ Calculations represent line coverage. Results may change as calculations are updated. Updated |
@codex review |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
User trial at Locally (macOS, bash 5.3): I ran
I also ran In CI at this head: run 43072881971 passed Repeat run: I re-ran Not proven: whether the original failure was the listing lag. If it was, the re-read absorbs it. If it was something else, the next failure will now name the check that broke. "Stays green across repeated runs" rests on the two green ordinary runs here plus the runs on 🤖 Generated with Claude Code |
Why
A required test on the main branch failed once with no message, so nobody could tell which of its checks broke or whether it was a real regression. The likeliest cause is that it reads the list of uploaded files a few seconds too early.
What
Each check in that test now says what it expected and what it found when it fails. The list of uploaded files is read again for a short, bounded time while it is still incomplete; a missing or extra file still fails.
Fixes #904
🤖 Generated with Claude Code