-
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 #548
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 #548 +/- ##
=======================================
Coverage 97.51% 97.51%
=======================================
Files 420 420
Lines 35407 35407
=======================================
Hits 34526 34526
Misses 881 881
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 #548 +/- ##
=======================================
Coverage 97.51% 97.51%
=======================================
Files 420 420
Lines 35407 35407
=======================================
Hits 34526 34526
Misses 881 881
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 #548 +/- ##
=======================================
Coverage 97.51% 97.51%
=======================================
Files 420 420
Lines 35407 35407
=======================================
Hits 34526 34526
Misses 881 881
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 #548 +/- ##
=======================================
Coverage 97.53% 97.53%
=======================================
Files 451 451
Lines 36129 36129
=======================================
Hits 35238 35238
Misses 891 891
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 |
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.