-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix: only process flakes for head of pull #545
Conversation
instead of processing all the commits on a pull that was merged, it's more correct to process only the testinstances that occurred on the head of the pull, since we can only guarantee that the test instances on the head of the pull are reflective of the state of the code on main.
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #545 +/- ##
==========================================
- Coverage 97.51% 97.51% -0.01%
==========================================
Files 420 420
Lines 35432 35429 -3
==========================================
- Hits 34550 34547 -3
Misses 882 882
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #545 +/- ##
==========================================
- Coverage 97.51% 97.51% -0.01%
==========================================
Files 420 420
Lines 35432 35429 -3
==========================================
- Hits 34550 34547 -3
Misses 882 882
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found @@ Coverage Diff @@
## main #545 +/- ##
==========================================
- Coverage 97.51% 97.51% -0.01%
==========================================
Files 420 420
Lines 35432 35429 -3
==========================================
- Hits 34550 34547 -3
Misses 882 882
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #545 +/- ##
==========================================
- Coverage 97.53% 97.53% -0.01%
==========================================
Files 451 451
Lines 36155 36152 -3
==========================================
- Hits 35263 35260 -3
Misses 892 892
Flags with carried forward coverage won't be shown. Click here to find out more.
This change has been scanned for critical changes. Learn more |
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.
Changes make sense to me
instead of processing all the commits on a pull that was merged, it's more correct to process only the testinstances that occurred on the head of the pull, since we can only guarantee that the test instances on the head of the pull are reflective of the state of the code on main.