Skip to content

ci: bump codecov-action to v6 and fail CI on upload error#690

Merged
Intron7 merged 1 commit into
scverse:mainfrom
timtreis:fix/codecov-action-v6-signature
Jun 8, 2026
Merged

ci: bump codecov-action to v6 and fail CI on upload error#690
Intron7 merged 1 commit into
scverse:mainfrom
timtreis:fix/codecov-action-v6-signature

Conversation

@timtreis

@timtreis timtreis commented Jun 8, 2026

Copy link
Copy Markdown
Member

Two changes to every codecov upload step in this repo:

  1. codecov-action -> @v6. Versions below 6.0.2 verify the codecov CLI binary against Codecov's old Keybase key, which they bricked after migrating accounts, so uploads now fail with Could not verify signature. @v6 (= 6.0.2) ships the updated key.
  2. fail_ci_if_error: true. Matches the cookiecutter-scverse template default so a broken upload fails CI loudly instead of passing silently.

Ref: codecov/codecov-action#1956

codecov-action below v6.0.2 verifies the codecov CLI against Codecov's
old Keybase key, which they bricked after migrating accounts, so uploads
now fail with "Could not verify signature". v6 (= 6.0.2) ships the
updated key.

Also sets fail_ci_if_error: true (the cookiecutter-scverse template
default) so a broken coverage upload surfaces as a CI failure instead of
passing silently.

Ref: codecov/codecov-action#1956

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Intron7 Intron7 enabled auto-merge (squash) June 8, 2026 15:54
@codecov-commenter

codecov-commenter commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.29%. Comparing base (ca6f571) to head (1296987).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #690   +/-   ##
=======================================
  Coverage   90.29%   90.29%           
=======================================
  Files         104      104           
  Lines        8964     8964           
=======================================
  Hits         8094     8094           
  Misses        870      870           

@Intron7 Intron7 merged commit 8bbdf79 into scverse:main Jun 8, 2026
22 of 26 checks passed
@timtreis timtreis deleted the fix/codecov-action-v6-signature branch June 8, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants