-
Notifications
You must be signed in to change notification settings - Fork 10
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
add True as default for wait_for_ci #533
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Changes have been made to critical files, which contain lines commonly executed in production. Learn more ✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #533 +/- ##
=======================================
Coverage 97.50% 97.50%
=======================================
Files 449 449
Lines 35739 35739
=======================================
Hits 34848 34848
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 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #533 +/- ##
=======================================
Coverage 97.48% 97.48%
=======================================
Files 418 418
Lines 35016 35016
=======================================
Hits 34135 34135
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 #533 +/- ##
=======================================
Coverage 97.48% 97.48%
=======================================
Files 418 418
Lines 35016 35016
=======================================
Hits 34135 34135
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 #533 +/- ##
=======================================
Coverage 97.48% 97.48%
=======================================
Files 418 418
Lines 35016 35016
=======================================
Hits 34135 34135
Misses 881 881
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Fixes codecov/feedback#440 |
@nora-codecov Was this one supposed to close? |
@drazisil-codecov fluke with merge queue - sus 🤨 |
The way it is written, if
wait_for_ci
is not supplied, it evaluates asFalse-y
per our docs,
wait_for_ci
default isTrue
This changes the evaluation, so if
wait_for_ci
is not supplied, it evaluates toTrue
which follows our intended behavior.