👷 [ci] (coverage) Preserve repository paths in Codecov reports - #10
Conversation
Generate Cobertura filenames relative to the repository, disable provider-side report expiry, and keep the existing 75 percent coverage gate intact. Add the copyright notices required by the current Ruff rules so a fresh CI dependency install can reach the coverage upload instead of failing during lint.
|
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #10 +/- ##
=======================================
Coverage ? 80.68%
=======================================
Files ? 6
Lines ? 994
Branches ? 141
=======================================
Hits ? 802
Misses ? 143
Partials ? 49
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 92c32a7dfe
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| branch = true | ||
| source = [ "skills/github-manage-security-alerts/scripts" ] | ||
| branch = true | ||
| include = [ "skills/github-manage-security-alerts/scripts/*.py" ] |
There was a problem hiding this comment.
Preserve source discovery in the coverage gate
Replacing source with include weakens the coverage run used by the “Run coverage tests” step in .github/workflows/python-tests.yml: coverage.py uses include only to filter files it observed, whereas source also discovers eligible files that were never executed. If a new production module under this scripts directory is not imported by any test, it will now be omitted entirely instead of contributing zero-covered lines, allowing the 75% gate to pass despite the missing tests. Retain source discovery (for example from the repository root) and apply the production-path filter at report time so repository-relative XML paths are preserved without shrinking the denominator.
Useful? React with 👍 / 👎.



Summary
Fix Codecov provider processing for the Python security-alert skill report without weakening its coverage gate.
Changes
Validation