Skip to content

Ci fix alt - #677

Open
xdelaruelle wants to merge 5 commits into
envmodules:mainfrom
xdelaruelle:ci-fix-alt
Open

Ci fix alt#677
xdelaruelle wants to merge 5 commits into
envmodules:mainfrom
xdelaruelle:ci-fix-alt

Conversation

@xdelaruelle

Copy link
Copy Markdown
Collaborator

No description provided.

This reverts commit 87deb87.

Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
The no-token upload path lets Codecov verify a run through a
GitHub Actions identity token, but codecov-action only requests
that token when its own opt-in input for it is enabled, and the
job running the step needs the id-token permission to obtain the
token in the first place. Neither was set, so the action fell
back to a legacy PR-label identification method that Codecov
rejects with a "Token required" error. Grant id-token: write and
turn on that opt-in input on the jobs that upload coverage.

Assisted-by: Claude:claude-sonnet-5
Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
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