Skip to content

[codex] Wire CI coverage upload to Codecov token#24

Merged
erikhoward merged 1 commit intomainfrom
codex/ci-codecov-coverage
Feb 15, 2026
Merged

[codex] Wire CI coverage upload to Codecov token#24
erikhoward merged 1 commit intomainfrom
codex/ci-codecov-coverage

Conversation

@erikhoward
Copy link
Member

Summary

This PR updates CI coverage publishing so Go coverage generated in GitHub Actions is uploaded to Codecov using the existing CODECOV_TOKEN secret.

Problem

Coverage generation already existed in CI, but Codecov uploads were not explicitly wired to the token secret. In token-required configurations this can cause uploads to be skipped or fail silently.

Root Cause

The workflow used codecov/codecov-action without passing CODECOV_TOKEN from GitHub Secrets, and upload failures were not configured to fail the job.

Changes

  • In .github/workflows/ci.yml:
    • Added job env wiring: CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}.
    • Added artifact upload step for coverage.out on Ubuntu (actions/upload-artifact@v4).
    • Updated Codecov upload step to:
      • run only when token is present,
      • pass token: ${{ env.CODECOV_TOKEN }},
      • set fail_ci_if_error: true.

Validation

  • Workflow file changes validated locally by review and commit.
  • No Go source behavior changed.

@erikhoward erikhoward marked this pull request as ready for review February 15, 2026 07:41
@erikhoward erikhoward merged commit ab1ff65 into main Feb 15, 2026
6 checks passed
@erikhoward erikhoward deleted the codex/ci-codecov-coverage branch February 15, 2026 07:41
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