Skip to content

Add GitHub native coverage comments for PRs#606

Draft
Copilot wants to merge 8 commits intomasterfrom
copilot/fix-codecov-issues
Draft

Add GitHub native coverage comments for PRs#606
Copilot wants to merge 8 commits intomasterfrom
copilot/fix-codecov-issues

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

Coverage comments were not appearing on PRs because the postprocess workflow (which uploads to Codecov) only runs for master branch pushes via workflow_run trigger.

Solution

Hybrid coverage approach:

  • PRs: GitHub native coverage comment in CI pipeline
  • Master: Existing Codecov upload in postprocess (unchanged)

Changes

.github/workflows/ci_pipeline.yml - Added coverage_comment_on_pr job:

  • Runs only for pull_request events after all test jobs complete
  • Downloads coverage artifacts from test matrix jobs
  • Combines coverage and posts comment via py-cov-action/python-coverage-comment-action@v3
  • Uses built-in github.token, no external service dependency
coverage_comment_on_pr:
  if: github.event_name == 'pull_request' && !cancelled()
  needs: [user_cpu_tests_linux, project_cpu_tests_linux, ...]
  permissions:
    pull-requests: write

Benefits:

  • Immediate PR coverage feedback without external service
  • Codecov retained for master branch historical tracking and badge generation
  • Simpler token management

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 7, 2026 14:12
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Copilot AI and others added 3 commits February 7, 2026 14:14
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Co-authored-by: pancetta <7158893+pancetta@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code coverage not showing for PRs Add GitHub native coverage comments for PRs Feb 7, 2026
Copilot AI requested a review from pancetta February 7, 2026 14:22
@github-actions
Copy link

github-actions bot commented Feb 7, 2026

Coverage report

This PR does not seem to contain any modification to coverable code.

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.

2 participants