From 8296c4cab088920c7453e0bab14952d913a4b7ac Mon Sep 17 00:00:00 2001 From: Colin Jermain Date: Tue, 28 Oct 2025 21:50:59 -0400 Subject: [PATCH 1/2] Adding CODECOV_TOKEN secret --- .github/workflows/CI.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 5145915..de61974 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -44,5 +44,6 @@ jobs: cargo llvm-cov report --lcov --output-path coverage.lcov - uses: codecov/codecov-action@v5 with: + token: ${{ secrets.CODECOV_TOKEN }} files: coverage.lcov,coverage.xml name: ${{ matrix.os }} From bd281d9f8eb43693fab031fd1a2ede208f88a192 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 29 Oct 2025 01:54:15 +0000 Subject: [PATCH 2/2] Document CODECOV_TOKEN requirement in README MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add note explaining that codecov-action@v4+ requires a CODECOV_TOKEN for uploading coverage reports. Includes reference to authoritative Codecov documentation for token setup. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- README.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/README.md b/README.md index 95b879c..c5265a0 100644 --- a/README.md +++ b/README.md @@ -99,6 +99,11 @@ The last step of this process happens in the CI, where we upload both coverage files to CodeCov. [Merging reports](https://docs.codecov.com/docs/merging-reports) is an automatic feature of CodeCov, so the final view shows the combined view. +**Note:** Starting with `codecov-action@v4`, a `CODECOV_TOKEN` is required for +uploading coverage reports. Add your token to GitHub repository secrets (see +[Adding the Codecov token](https://docs.codecov.com/docs/adding-the-codecov-token)) +and reference it in the workflow with `token: ${{ secrets.CODECOV_TOKEN }}`. + # Excluding code from coverage The `#[pymodule]` initialization function in `src/lib.rs` is surrounded by