Skip to content

Authenticate the Codecov upload - #215

Merged
edridudi merged 2 commits into
mainfrom
claude/codecov-token
Sep 13, 2026
Merged

edridudi merged 2 commits into
mainfrom
claude/codecov-token

Conversation

@edridudi

Copy link
Copy Markdown
Collaborator

Coverage uploads have been failing since 2026-09-12. Codecov's newest data is commit e6922b5; everything merged since — including #213 and #214 — never reached it. That is why the badge still reads 69% after the selector fix.

From the most recent run's log:

['error'] There was an error running the uploader: Error uploading to https://codecov.io:
429 - {"message":"Rate limit reached. Please upload with the Codecov repository upload
token to resolve issue. Expected time to availability: 1291s."}

['info'] Codecov will exit with status code 0.

Two separate problems:

Tokenless uploads from public repos are rate limited. Codecov's own message names the fix: use a repository upload token.

The failure was invisible. codecov-action exits 0 on upload errors, so the job showed a green tick while the badge quietly went stale for two days and several merges.

Change

with:
  token: ${{ secrets.CODECOV_TOKEN }}
  fail_ci_if_error: true

fail_ci_if_error matters as much as the token — without it the next upload problem is equally silent.

CODECOV_TOKEN is already set on the repo.

Expected effect

The badge should move from 69% to ~82% on the first successful upload from main, reflecting the 175 tests #214 brought into the job.

Not addressed here

codecov-action is on v3 while v5 is current, and the run warns that actions/cache@v3, actions/checkout@v2 and actions/setup-java@v2 are being forced onto Node 24 across all workflows. Worth a separate pass.

🤖 Generated with Claude Code

dudiedri and others added 2 commits September 14, 2026 01:20
Uploads have been failing since 2026-09-12:

  429 - {"message":"Rate limit reached. Please upload with the Codecov
  repository upload token to resolve issue."}

Tokenless uploads from public repos are rate limited. codecov-action exits
0 on upload errors, so the job kept reporting success while Codecov's data
went stale - the badge has been showing coverage from two days and several
merges ago, including none of the work in #213 or #214.

Passes CODECOV_TOKEN, and sets fail_ci_if_error so a failed upload is
visible rather than silent. The silence is what let this go unnoticed.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The v3 action bundles uploader 0.8.0, which only accepts the legacy 36
character UUID token and rejects today's 50 character tokens:

  Token found by environment variables with length 50 did not pass validation

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@edridudi
edridudi merged commit 8500249 into main Sep 13, 2026
3 of 4 checks passed
@codecov

codecov Bot commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.83%. Comparing base (5a617dc) to head (c3c33a2).
⚠️ Report is 17 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #215      +/-   ##
============================================
+ Coverage     69.39%   77.83%   +8.44%     
- Complexity      246      298      +52     
============================================
  Files            37       37              
  Lines           941      961      +20     
  Branches         73       79       +6     
============================================
+ Hits            653      748      +95     
+ Misses          249      175      -74     
+ Partials         39       38       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@edridudi
edridudi deleted the claude/codecov-token branch September 13, 2026 22:31
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.

1 participant