ci(tests): run tests via reusable python-ci workflow#89
Conversation
Replace the inline Test Suite job with the org-standard netresearch/.github python-ci.yml@main reusable (unit + integration pytest with coverage, Codecov upload). Removes the per-repo codecov-action reference that Renovate kept bumping (#88); the action is now pinned once in netresearch/.github. Coverage uploads once (ubuntu/3.14) instead of from every matrix cell. The lint, security, build, docs and integration-e2e jobs are unchanged; build/integration-e2e still depend on the 'test' job by name. Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
Dependency ReviewThe following issues were found:
License Issues.github/workflows/ci.yml
OpenSSF Scorecard
Scanned Files
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #89 +/- ##
==========================================
- Coverage 69.47% 68.87% -0.60%
==========================================
Files 22 22
Lines 3345 3345
==========================================
- Hits 2324 2304 -20
- Misses 1021 1041 +20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s CI pipeline by replacing the inline test job implementation with the org-standard reusable workflow netresearch/.github/.github/workflows/python-ci.yml@main, keeping the test job identifier intact so existing needs: [test] dependencies continue to work.
Changes:
- Replaced the inline matrix-based pytest + coverage + Codecov upload job with the reusable
python-ci.ymlworkflow. - Configured the reusable workflow to run unit + integration tests across
ubuntu-latest,macos-latest,windows-lateston Python3.14. - Adjusted coverage upload behavior to upload to Codecov only once (ubuntu / 3.14) instead of from every matrix cell.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
What
Replaces the inline
Test Suitejob with the org-standardnetresearch/.github/.github/workflows/python-ci.yml@mainreusable (added in netresearch/.github#174). It runs the unit + integration pytest suites with coverage and uploads to Codecov.lint,security,build,docs, andintegration-e2eare unchanged. The reusable caller keeps the job nametest, so theneeds: [lint, test]/needs: [test]dependencies stay valid.Why
Renovate was bumping
codecov/codecov-actionin this repo per release (#88). The action now lives once inpython-ci.ymland is pinned/bumped centrally — the same model PHP repos use viaphp-ci.yml. There is no longer an inlinecodecov-actionreference here.Closes #88.
Behavior notes
ubuntu-latest,macos-latest,windows-lateston Python 3.14.name=values.pip install -r requirements-dev.txt(matching the previous job; not switched to uv).Residual
The
lint,security,build,docs,integration-e2ejobs still pinactions/checkout/actions/setup-python, which Renovate will still bump occasionally. This PR stops the codecov churn specifically.Validation
yamllint(org config) andactionlintclean.