-
-
Notifications
You must be signed in to change notification settings - Fork 344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider integrating the covdefaults
plugin
#3163
Comments
covdefaults
plugin, as it injects most of these automatically. The only thing you'd need to override additionally is the fail_under
setting, which it sets to 100%, but it's not compatible with runtime-dependent tests that cannot reach 100% under one specific runtime. So it's good to set it to something sufficiently low while requiring 100% in Codecov.covdefaults
plugin, as it injects most of these automatically
covdefaults
plugin, as it injects most of these automaticallycovdefaults
plugin
https://github.com/asottile/covdefaults looks handy as it configures a bunch of nice settings I'm not sure I agree with all the no-cov pragmas, however. |
Ooh, the platform/version pragmas are a really interesting way of enforcing 100% coverage for each individual run. Maybe not for trio, but will consider it for projects which have a very small amount of platform/version-dependent code. This should be added to upstream coverage tbh But yeah I'm not finding a way of removing some of the |
uvicorn uses an interesting approach: https://github.com/encode/uvicorn/blob/master/pyproject.toml#L124 |
I also found nedbat/coveragepy#1699 which mentioned https://pypi.org/project/coverage-simple-excludes/ and https://github.com/beeware/briefcase/blob/a898beaed7126f07b3a0ae21df8b291d512f798f/pyproject.toml#L202 (which, like uvicorn, uses https://pypi.org/project/coverage-conditional-plugin/) |
Oh, nice! More projects to try out for me :) |
Originally posted by @webknjaz in #3159 (comment)
The text was updated successfully, but these errors were encountered: