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

Disable Codecov Check Run Annotations #2537

Conversation

eliatcodecov
Copy link
Contributor

Disables check run annotations for codecov in the codecov.yml. This should prevent the ''X line not covered by tests'' annotations in PRs .

Disables check run annotations for codecov in the codecov.yml. This should prevent the ''X line not covered by tests'' annotations in PRs .
@sentrivana sentrivana added the Trigger: tests using secrets PR code is safe; run CI label Nov 30, 2023
@github-actions github-actions bot removed the Trigger: tests using secrets PR code is safe; run CI label Nov 30, 2023
@sentrivana sentrivana added the Trigger: tests using secrets PR code is safe; run CI label Nov 30, 2023
@github-actions github-actions bot removed the Trigger: tests using secrets PR code is safe; run CI label Dec 1, 2023
@antonpirker antonpirker added the Trigger: tests using secrets PR code is safe; run CI label Dec 4, 2023
@github-actions github-actions bot removed the Trigger: tests using secrets PR code is safe; run CI label Dec 4, 2023
@sentrivana
Copy link
Contributor

Leaving this unmerged for now since I'd like to see if the annotations experience improves after we've merged the CI overhaul in #2567 -- it affects how our coverage reports are generated so maybe something changes.

@antonpirker
Copy link
Member

Will merge this now, because it makes our lives easier :-)

@antonpirker antonpirker enabled auto-merge (squash) April 4, 2024 09:23
@antonpirker antonpirker self-assigned this Apr 4, 2024
@antonpirker antonpirker merged commit 0683552 into getsentry:master Apr 4, 2024
124 checks passed
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