Skip to content

Fix codecov uploads: declare the test environment; README wording - #60

Merged
tilo merged 2 commits into
mainfrom
fix-codecov--1.1.0
Aug 7, 2026
Merged

Fix codecov uploads: declare the test environment; README wording#60
tilo merged 2 commits into
mainfrom
fix-codecov--1.1.0

Conversation

@tilo

@tilo tilo commented Aug 7, 2026

Copy link
Copy Markdown
Owner
  • Declare environment: test on the test matrix job — the CODECOV_TOKEN secret lives in that GitHub environment, and environment secrets are only injected into jobs that declare the environment. Until now every upload ran with an empty token (Token length: 0 in the logs) and codecov rejected it ("Token required because branch is protected") — which is why the badge shows "unknown" and codecov has zero processed reports.
  • README wording: "resource" instead of "place"; add "assignments" to the use cases.
    After merging, the first push-run on main should produce the first processed codecov report. Also set the default branch to main in codecov's repo settings — its record still says master (it synced minutes before the GitHub flip), and the badge follows codecov's default branch.

tilo and others added 2 commits August 7, 2026 10:42
The CODECOV_TOKEN secret lives in that GitHub environment; environment secrets are only injected into jobs that declare the environment, so uploads ran with an empty token ("Token length: 0") and codecov rejected them ("Token required because branch is protected").

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ShKZCYJFU7bdDSvRT2FgN1
@tilo
tilo merged commit cff4e03 into main Aug 7, 2026
19 checks passed
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

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