Skip to content

fix(ci): switch Codecov upload from OIDC to token authentication (#96) - #162

Merged
Romuald Lemesle (RomuDeuxfois) merged 1 commit into
mainfrom
fix/codecov-use-token
Jul 21, 2026
Merged

fix(ci): switch Codecov upload from OIDC to token authentication (#96)#162
Romuald Lemesle (RomuDeuxfois) merged 1 commit into
mainfrom
fix/codecov-use-token

Conversation

@RomuDeuxfois

Copy link
Copy Markdown
Member

Summary

  • Replace use_oidc: true with token: ${{ secrets.CODECOV_TOKEN }} in the coverage upload step
  • Remove id-token: write permission (no longer needed)

OIDC authentication is not configured on the Codecov organization. This switches to token-based authentication, aligned with how openaev handles Codecov uploads.

Prerequisite: the CODECOV_TOKEN secret must be set at the org or repo level.

Copilot AI review requested due to automatic review settings July 21, 2026 10:05
@github-actions github-actions Bot added the filigran team Item from the Filigran team. label Jul 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Switches the GitHub Actions Codecov upload in the implant CI workflow from OIDC-based authentication to a repository/org secret token, matching the stated Codecov org configuration (OIDC not enabled) and removing no-longer-needed permissions.

Changes:

  • Replaced use_oidc: true with token: ${{ secrets.CODECOV_TOKEN }} for the Codecov upload step.
  • Removed the workflow’s id-token: write permission since OIDC is no longer used.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RomuDeuxfois
Romuald Lemesle (RomuDeuxfois) merged commit dbd4f0d into main Jul 21, 2026
23 checks passed
@RomuDeuxfois
Romuald Lemesle (RomuDeuxfois) deleted the fix/codecov-use-token branch July 21, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants