Adds test coverage check in repo (#68) #752
Conversation
* Adds test coverage check in repo * Adds pytest covergae dependency * Update threshold for new code
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 ☂️ |
|
@martindurant Can you take a quick look, if its okay with you to add test coverage check in the ci, would make remaining changes in repo and merge this PR |
| - name: Upload coverage to Codecov | ||
| uses: codecov/codecov-action@v5 | ||
| with: | ||
| token: ${{ secrets.CODECOV_TOKEN }} |
There was a problem hiding this comment.
Have you created this secret?
There was a problem hiding this comment.
not yet, was waiting for you to LGTM this PR ( in case you had any apprehensions for codecov). I ll also add codecov github app
There was a problem hiding this comment.
Can you please add me in users in codecov settings. I dont have permission to see the access token
There was a problem hiding this comment.
How do I do that?
I do already see some data at https://app.codecov.io/github/fsspec/gcsfs/commits
3 participants
Adds automated code coverage tracking to maintain testing standards and prevent regression in future contributions.
Updates the CI pipeline to generate XML reports using
pytest-covand uploads them toCodecovfor immediate visibilityImplements a strict codecov.yml policy that fails the build if test coverage declines, ensuring all new code is adequately tested