ci: drop the unused CODECOV_TOKEN reference - #28
Merged
Conversation
Codecov accepts tokenless uploads from public repositories, so the secret
reference was never needed. Confirmed against a known-working public-repo
configuration, which uploads and reports with no token input at all.
This is a correction to the config, not a fix for the failed upload. An
empty `token:` input and an absent one are the same thing to the action —
it checks `[ -n "$INPUT_TOKEN" ]` and skips when empty, which is exactly
what the failing run did ("Token length: 0"). Removing the line changes
no behavior; it stops the workflow from pointing at a secret that does
not exist and should not need to.
The upload that failed was rejected because the repository is not
activated on Codecov's side ("active": false, "activated": false via
their API), not because it lacked credentials. Their error string for
that case — "Token required - not valid tokenless upload" — reads like an
auth problem and sent me down the wrong path. Activation is a
console-side action; nothing in this repository can do it.
patramsey
force-pushed
the
ci/codecov-tokenless
branch
from
August 2, 2026 19:30
66d01c1 to
e419088
Compare
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
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.
Codecov accepts tokenless uploads from public repositories, so the secret reference was never needed. Verified against a known-working public-repo configuration, which uploads and reports with no token input at all:
This is a correction, not a fix
Worth being clear so nobody expects the badge to light up from this PR: an empty
token:input and an absent one are identical to the action. It checks[ -n "$INPUT_TOKEN" ]and skips when empty — which is precisely what the failing run did (-> Token length: 0). Removing the line changes no behavior. It just stops the workflow pointing at a secret that doesn't exist and shouldn't need to.What actually blocked the upload
This repository isn't activated on Codecov's side. Their API for it:
{ "active": false, "activated": false, "totals": null }Codecov's error for that case is
{"message":"Token required - not valid tokenless upload"}, which reads like an auth failure and is what sent me looking for a missing credential. The token was a red herring — activation is the discriminator, and a public repository that is activated uploads fine with no token at all.Activation is a console-side action — nothing in this repo can do it. Once
patramsey/namecom-cliis activated at codecov.io, uploads should start landing with this config as-is, and the README badge flips fromunknownto ~64%.Residual uncertainty
The configuration I verified tokenless behavior against runs codecov-action v5.5.5; this repo runs v7.0.0. I've confirmed v5 uploads tokenless on a push to a default branch, but not v7 — its own logic only sets the tokenless variable when
CC_FORK == 'true', which may be an internal detail or may matter server-side. If uploads still don't land after activation, that version difference is the first thing to check.Test plan
ci.ymlparses as valid YAMLtoken:line removed — no other workflow change