Skip to content

Publish coverage from one matrix leg - #187

Merged
rdlauer merged 1 commit into
mainfrom
fix-coveralls-race
Sep 23, 2026
Merged

rdlauer merged 1 commit into
mainfrom
fix-coveralls-race

Conversation

@rdlauer

@rdlauer rdlauer commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

coveralls --service=github derives its job id from GITHUB_RUN_ID, which is identical
across all five matrix legs. Five submissions claim the same job for the same commit,
Coveralls accepts one and 422s the rest, and the non-zero exit fails the build.

Seen on #185:

Could not submit coverage: 422 Client Error: Unprocessable Entity
  for url: https://coveralls.io/api/v1/jobs
##[error]Process completed with exit code 1.

All 419 tests had passed; only the publish step failed. It's a race, so it bites
intermittently — whichever leg loses.

Coverage is identical across interpreter versions, so publishing from 3.12 alone
removes the collision and loses nothing.

🤖 Generated with Claude Code

`coveralls --service=github` derives its job id from GITHUB_RUN_ID, which is the
same for all five matrix legs. Five submissions claim the same job for the same
commit, Coveralls accepts one and 422s the rest, and the step's non-zero exit
fails the build -- intermittently, whichever leg loses the race (PR #185, 3.11).

Coverage is identical across interpreter versions, so publishing from 3.12 alone
loses nothing and removes the collision.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@rdlauer
rdlauer requested a review from Bucknalla September 23, 2026 15:31
@rdlauer
rdlauer merged commit c0dc040 into main Sep 23, 2026
7 checks passed
@rdlauer
rdlauer deleted the fix-coveralls-race branch September 23, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants