Skip to content

ci: enable codecov for Python test coverage#1575

Open
ramakrishnap-nv wants to merge 1 commit into
mainfrom
ci/enable-codecov
Open

ci: enable codecov for Python test coverage#1575
ramakrishnap-nv wants to merge 1 commit into
mainfrom
ci/enable-codecov

Conversation

@ramakrishnap-nv

@ramakrishnap-nv ramakrishnap-nv commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Removes the explicit run_codecov: false override, adds codecov.yml (patch checks informational, never blocks), suppresses spurious coverage.py warnings, and excludes codecov.yml from changed-files triggers.

Requires CODECOV_TOKEN to be set as a repo/org secret.

Removes the explicit `run_codecov: false` override in pr.yaml and
test.yaml, restoring the shared-workflow default (true). Adds
codecov.yml at repo root with patch checks set to informational so
coverage never blocks a PR. Adds disable_warnings to .coveragerc to
suppress spurious coverage.py noise. Excludes codecov.yml from
changed-files triggers so a config-only edit does not re-run CI.

Note: requires CODECOV_TOKEN to be set as a repo or org secret.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
@ramakrishnap-nv ramakrishnap-nv requested review from a team as code owners July 14, 2026 19:48
@ramakrishnap-nv ramakrishnap-nv self-assigned this Jul 14, 2026
@ramakrishnap-nv ramakrishnap-nv added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 42a60fc1-8082-4856-8855-0de66a559401

📥 Commits

Reviewing files that changed from the base of the PR and between cd8ddad and 003e569.

📒 Files selected for processing (4)
  • .github/workflows/pr.yaml
  • .github/workflows/test.yaml
  • codecov.yml
  • python/.coveragerc
💤 Files with no reviewable changes (1)
  • .github/workflows/test.yaml

📝 Walkthrough

Walkthrough

Changes

Codecov CI integration

Layer / File(s) Summary
Coverage configuration
codecov.yml, python/.coveragerc
Adds Codecov status and annotation settings, a patch coverage threshold, and suppression for selected coverage warnings.
Workflow coverage integration
.github/workflows/pr.yaml, .github/workflows/test.yaml
Excludes codecov.yml from unrelated change groups and removes explicit Codecov suppression from conda Python test jobs.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: enabling Codecov for Python test coverage.
Description check ✅ Passed The description accurately matches the changes in the PR and explains the Codecov setup and related workflow updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/enable-codecov

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

CI Test Summary

8 failed · 10 passed · 3 skipped

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant