Skip to content
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

Merged
merged 1 commit into from
Apr 6, 2017
Merged

Update codecov config #3906

merged 1 commit into from
Apr 6, 2017

Conversation

ruflin
Copy link
Collaborator

@ruflin ruflin commented Apr 4, 2017

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.

@ruflin
Copy link
Collaborator Author

ruflin commented Apr 5, 2017

jenkins, retest it

Copy link
Contributor

@tsg tsg left a 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
Copy link
Contributor

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% :)

Copy link
Collaborator Author

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
Copy link
Contributor

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?

Copy link
Collaborator Author

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.
@ruflin ruflin force-pushed the codecov-adjustments branch from d1e802e to 3df95f2 Compare April 5, 2017 09:18
@ruflin
Copy link
Collaborator Author

ruflin commented Apr 5, 2017

Update version pushed.

@ruflin
Copy link
Collaborator Author

ruflin commented Apr 6, 2017

jenkins, retest it

@tsg tsg merged commit fcc02f8 into elastic:master Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants