-
Notifications
You must be signed in to change notification settings - Fork 299
Add codecov configuration #1002
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
Conversation
Codecov Report
Merging #1002 (71a609b) into master (c4337f1) will not change coverage.
The diff coverage isn/a.
❗ Current head 71a609b differs from pull request most recent head 0eb44af. Consider uploading reports for the commit 0eb44af to get more accurate results
Impacted file tree graph
@@ Coverage Diff @@ ## master #1002 +/- ## ======================================= Coverage 96.84% 96.84% ======================================= Files 65 65 Lines 3925 3925 ======================================= Hits 3801 3801 Misses 124 124
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered by Codecov. Last update c4337f1...0eb44af. Read the comment docs.
* Ensure that PR covers 100% of diff * Disable project coverage as it may be misleading * Disable github comment to avoid notifications
71a609b to
0eb44af
Compare
Uh oh!
There was an error while loading. Please reload this page.
Description of the Change
Once this is merged we can make codecov/patch a requirement so PR won't be mergeable if diff is not covered. In the rare cases where a code bit cannot be tested it should be marked as
# pragma: no coverin the code to make it clear.Checklist
CHANGELOG.mdupdated (only for user relevant changes)AUTHORS