Skip to content

Show coverage in the job summary and warn on a rejected Codecov upload - #14

Merged
imanimanyara merged 2 commits into
mainfrom
ci/coverage-summary
Oct 2, 2026
Merged

imanimanyara merged 2 commits into
mainfrom
ci/coverage-summary

Conversation

@imanimanyara

Copy link
Copy Markdown
Member
  • Add a stdlib coverage summary renderer for CI job summaries

Renders total, the fail_under floor read through coverage.py's own config, and the lowest-covered files from coverage.xml as Markdown, appending to $GITHUB_STEP_SUMMARY when set. It refuses a missing or empty report so a run that measured nothing never renders as clean. No third-party action is needed to see coverage on a run.

  • Show coverage in the job summary and warn on a rejected Codecov upload

Codecov has been rejecting every upload (repository not activated, token mismatch) and nothing said so: with fail_ci_if_error false the action exits 0 whatever the server answers. The canonical CI leg and the weekly baseline now write the coverage summary, keep coverage.xml as a 7-day artifact, and run the upload with fail_ci_if_error true under continue-on-error, so an HTTP 4xx fails the step without failing the job and a follow-up step raises a warning annotation.

Renders total, the fail_under floor read through coverage.py's own config, and the lowest-covered files from coverage.xml as Markdown, appending to $GITHUB_STEP_SUMMARY when set. It refuses a missing or empty report so a run that measured nothing never renders as clean. No third-party action is needed to see coverage on a run.
Codecov has been rejecting every upload (repository not activated, token mismatch) and nothing said so: with fail_ci_if_error false the action exits 0 whatever the server answers. The canonical CI leg and the weekly baseline now write the coverage summary, keep coverage.xml as a 7-day artifact, and run the upload with fail_ci_if_error true under continue-on-error, so an HTTP 4xx fails the step without failing the job and a follow-up step raises a warning annotation.
Copilot AI balanced review requested due to automatic review settings October 2, 2026 17:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@imanimanyara
imanimanyara merged commit 4372cb5 into main Oct 2, 2026
15 checks passed
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