Fix silent codecov upload failures by authenticating with a repo token - #52
Merged
Conversation
Every recent push to master's Test coverage step has been hitting Codecov's HTTP 429 rate limit for tokenless uploads (confirmed in the job logs: "Rate limit reached. Please upload with the Codecov repository upload token to resolve issue."), so coverage has silently stopped reaching codecov.io even though the CI step reports success (covr::codecov() doesn't fail the step on a bad HTTP response). Passes CODECOV_TOKEN from a repo secret, which covr::codecov() already knows to read. Requires adding a CODECOV_TOKEN secret (from codecov.io's repo settings) at github.com/sipss/GCIMS/settings/secrets/actions before this takes effect.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
master's "Test coverage" step has been hitting Codecov's HTTP 429 rate limit for tokenless uploads. The job logs show:"Rate limit reached. Please upload with the Codecov repository upload token to resolve issue."covr::codecov()doesn't fail the step on a bad HTTP response — it just prints the error body. So coverage uploads have been silently broken.CODECOV_TOKENfrom a repo secret to the step's environment.covr::codecov()already knows to read this env var to authenticate the upload.Requires a follow-up step outside this PR
This won't take effect until a
CODECOV_TOKENsecret exists on the repo. Get it from codecov.io (thesipss/GCIMSrepo's settings → upload token) and add it atgithub.com/sipss/GCIMS/settings/secrets/actions.Test plan
check-bioc.ymlparses as valid YAMLmaster(after theCODECOV_TOKENsecret is added) uploads to Codecov without a 429Generated by Claude Code