Skip to content

Commit

Permalink
ci: explicitly specify token for codecov
Browse files Browse the repository at this point in the history
Explicitly specify the codecov token to be used (i.e., `CODECOV_TOKEN`),
given that the latest v4 release of the codecov action requires it to be
able to generate coverage reports.
Additionally, fail CI if coverage reporting fails, since coverage is an
important enough metric for us to ensure that it's tracked consistently.
  • Loading branch information
thazhemadam committed Mar 1, 2024
1 parent 74ac3fa commit 1b1d190
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,3 +45,5 @@ jobs:
- uses: codecov/codecov-action@v4
with:
file: lcov.info
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true
2 changes: 2 additions & 0 deletions .github/workflows/Documentation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,5 @@ jobs:
- uses: codecov/codecov-action@v4
with:
file: lcov.info
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true
2 changes: 2 additions & 0 deletions .github/workflows/Downstream.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,3 +82,5 @@ jobs:
- uses: codecov/codecov-action@v4
with:
file: lcov.info
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true

0 comments on commit 1b1d190

Please sign in to comment.