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

chore(build): disable codecov status #776

Closed
wants to merge 2 commits into from

Conversation

kt3k
Copy link
Member

@kt3k kt3k commented Mar 2, 2021

This PR disables the commit status from codecov.io.

Codecov sends failed commit status to the branch when the coverage rate is decreased. Higher coverage rate is generally a good thing, but marking the commit as failure because of the decreased coverage doesn't make sense to our use case.

ref: https://docs.codecov.io/docs/commit-status#disabling-a-status

@caspervonb
Copy link
Contributor

Right so it shows up in the checks for the main branch which was not intentional.

Maybe we should set informational: true if we don't care about thresholds? (https://docs.codecov.io/docs/commit-status#informational

Use Codecov in informational mode. Default is false. If true is specified the resulting status will pass no matter what the coverage is or what other settings are specified. Informational mode is great to use if you want to expose codecov information to other developers in your pull request without necessarily gating PRs on that information.

@kt3k
Copy link
Member Author

kt3k commented Mar 2, 2021

Informational mode sounds good 👍 Updated the PR.

Copy link
Contributor

@caspervonb caspervonb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kt3k
Copy link
Member Author

kt3k commented Mar 2, 2021

This is landed as part of #775

@kt3k kt3k closed this Mar 2, 2021
@kt3k kt3k deleted the chore/disable-codecov-status branch March 2, 2021 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants