Conversation
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
There was a problem hiding this comment.
Pull request overview
Adds Codecov coverage reporting to the existing CI workflow to support issue #8 (test coverage tracking).
Changes:
- Upload
coverage.xmlto Codecov on Linux CI runs usingcodecov/codecov-action@v5.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| - name: Upload coverage to Codecov | ||
| if: runner.os == 'Linux' | ||
| uses: codecov/codecov-action@v5 | ||
| with: | ||
| token: ${{ secrets.CODECOV_TOKEN }} | ||
| files: ./coverage.xml | ||
| fail_ci_if_error: true |
There was a problem hiding this comment.
Don't think we care about this scenario
| with: | ||
| token: ${{ secrets.CODECOV_TOKEN }} | ||
| files: ./coverage.xml | ||
| fail_ci_if_error: true |
alexdewar
left a comment
There was a problem hiding this comment.
LGTM!
Could you add a status badge to the README while you're at it? Instructions are here.
|
This seems to have dropped of our collective radar over the holidays... I'll put the issue in the next sprint. |
Description
Adds codecov action to the Linux job. Secret was also added to the repo
Close #8
Type of change
Key checklist
python -m pytest)pre-commit run --all-files)Further checks
(Indicate issue here: # (issue))