Skip to content

ci: add coverage report on PRs#837

Open
radu-mocanu wants to merge 1 commit intomainfrom
chore/coverage-on-pr
Open

ci: add coverage report on PRs#837
radu-mocanu wants to merge 1 commit intomainfrom
chore/coverage-on-pr

Conversation

@radu-mocanu
Copy link
Copy Markdown
Collaborator

@radu-mocanu radu-mocanu commented May 8, 2026

Summary

  • coverage comment on PRs with 90% green / 80% orange thresholds via py-cov-action/python-coverage-comment-action
  • xml + html coverage artifacts (30-day retention)
  • coverage runs only on the ubuntu/3.13 matrix cell, no duplicate test runs
  • fix matrix python-version being silently ignored (since setup-python was using python-version-file which always pinned 3.11)

post coverage comment with 90% green / 80% orange thresholds, upload xml +
html artifacts. coverage runs only on the ubuntu/3.13 matrix cell. also
fix matrix python-version not being applied (was silently using 3.11 on
all cells via .python-version file).

- name: Coverage PR comment
if: "!contains(github.event.pull_request.labels.*.name, 'test-core-dev-version') && matrix.os == 'ubuntu-latest' && matrix.python-version == '3.13' && github.event_name == 'pull_request'"
uses: py-cov-action/python-coverage-comment-action@v3
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