-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
CI: Tweak our code coverage profile behavior #14967
Merged
Merged
Changes from 3 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
5bcfd75
Tweak our code coverage profile behavior
mattlord 14c677c
Increase runner size
mattlord 54082fd
Add covermode=atomic for parallel work
mattlord 9e8c768
More misc tweaks
mattlord 09f47dc
Move to the codecov action v4
mattlord a61bf8a
Fail fast for code coverage
mattlord a507a05
Ignore vttestserver related files
mattlord 06ebcba
Don't remove vttestserver
mattlord f3ea9e0
Restore deleted e2e test files before codecov work
mattlord 02eb2ac
Remove all e2e test files again
mattlord 9fc549e
Try going back to codecov-action@v3
mattlord 7a19e6d
Try to de-flake TestProbesPostDisable/aggregated
mattlord 72cb0ca
Revert "Try going back to codecov-action@v3"
mattlord a7861b9
Revert "Try to de-flake TestProbesPostDisable/aggregated"
mattlord 18f89bb
Merge remote-tracking branch 'origin/main' into codecov_tweaks
mattlord aa763f4
Count partial line coverage as a hit (line coverage)
mattlord 5c2a044
Add e2e test dirs to codecov ignore list
mattlord 56f139f
Ignore code generated files
mattlord a9850dd
Use new v4 GA action
mattlord 54639be
Merge remote-tracking branch 'origin/main' into codecov_tweaks
mattlord b5ae453
Fix TestGetDir unit test
mattlord 5e5bbd9
Try alternate ignore paths
mattlord c2fe9d2
Address review comment
mattlord File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
# https://docs.codecov.com/docs/ | ||
# https://docs.codecov.com/docs/codecov-yaml | ||
|
||
codecov: | ||
branch: main # Set the default branch | ||
|
||
comment: # https://docs.codecov.com/docs/pull-request-comments | ||
hide_project_coverage: false | ||
|
||
coverage: | ||
status: # https://docs.codecov.com/docs/commit-status | ||
project: | ||
default: | ||
informational: true # Don't ever fail the codecov/project or codecove/patch tests | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
just stating for posterity that i'm good with this given that https://docs.codecov.com/docs/github-checks#hiding-annotations-in-the-files-view exists
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.
Had no idea this existed, but tested and approved