-
Notifications
You must be signed in to change notification settings - Fork 767
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
Github actions coverage #5148
Github actions coverage #5148
Conversation
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.
Approving rust changes similar to #2644
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
... and 2693 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Jobs for coverage collections and upload to codecov for github PR's
Uses same test suit as test-linux-stable, splits tests into 5 parallel jobs, uploads to codecov, generates report comment and status checks (can be made required)
Codecov behavior (required coverage, thresholds, comment info etc.) is highly customizable via
.github/codecov.yaml
(reference)Unfortunately, some tests are excluded because with
-C instrument-coverage
they run very slowly, flaky, or fail (see nextest filter expression of test-linux-stable-coverage). So for now, this workflow is optional, and will only run for pr's with theGHA-coverage
label