-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Update codecov config #3906
Update codecov config #3906
Conversation
jenkins, retest it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, left two questions.
codecov.yml
Outdated
default: | ||
target: auto | ||
# Overall coverage should never drop more then 1% | ||
threshold: 0.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment above is wrong, says 1% :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
argh, updated the value and not the comment ...
target: auto | ||
threshold: 0.1 | ||
# Allows PRs without tests, overall stats count | ||
threshold: 100 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean this check will always be green, because threshlod is 100?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly. Patch level check will always be green. Otherwise if 1 code line is added which is not covered, it would have 0% coverage.
Codecov will report coverage for the patch and for the project based on the pull request. In case the overall coverage drops by more then 0.5% the pull request will fail. It is still possible to have pull requests without tests which will not fail because the overall coverage did not change too much.
d1e802e
to
3df95f2
Compare
Update version pushed. |
jenkins, retest it |
Codecov will report coverage for the patch and for the project based on the pull request. In case the overall coverage drops by more then 0.5% the pull request will fail. It is still possible to have pull requests without tests which will not fail because the overall coverage did not change too much.