Skip to content

ci(test): show coverage delta vs main in PR comment#65

Merged
benner merged 1 commit into
mainfrom
ci/add-coverage-delta-vs-main
May 1, 2026
Merged

ci(test): show coverage delta vs main in PR comment#65
benner merged 1 commit into
mainfrom
ci/add-coverage-delta-vs-main

Conversation

@benner
Copy link
Copy Markdown
Owner

@benner benner commented May 1, 2026

Add a coverage-baseline.yml workflow that runs on push to main and
uploads coverage.xml as the main-coverage artifact. The coverage-comment
workflow fetches the latest baseline run via the GitHub API (pure Python,
stdlib only), downloads the artifact, and computes the Δ to include in
the comment title (e.g. "Coverage Report (Δ +1.2%)"). Falls back to
"Coverage Report" when no baseline exists yet.

Add a coverage-baseline.yml workflow that runs on push to main and
uploads coverage.xml as the main-coverage artifact. The coverage-comment
workflow fetches the latest baseline run via the GitHub API (pure Python,
stdlib only), downloads the artifact, and computes the Δ to include in
the comment title (e.g. "Coverage Report (Δ +1.2%)"). Falls back to
"Coverage Report" when no baseline exists yet.

Signed-off-by: Nerijus Bendžiūnas <nerijus.bendziunas@gmail.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/git_commit_guard
   __init__.py345299%216, 602
TOTAL345299% 

@benner benner marked this pull request as ready for review May 1, 2026 04:00
@benner benner merged commit 854d672 into main May 1, 2026
4 checks passed
@benner benner deleted the ci/add-coverage-delta-vs-main branch May 1, 2026 04:00
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