-
Notifications
You must be signed in to change notification settings - Fork 168
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: Enable "fail on error" in CI #688
Conversation
Codecov Report
@@ Coverage Diff @@
## master #688 +/- ##
==========================================
+ Coverage 49.01% 49.02% +0.01%
==========================================
Files 325 325
Lines 16574 16574
Branches 7744 7744
==========================================
+ Hits 8124 8126 +2
+ Misses 3019 3017 -2
Partials 5431 5431
Continue to review full report at Codecov.
|
This is now ready for review. |
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.
See #706 for my comments.
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.
LGTM
This PR is taking #687 to its logical conclusion.
It should not be merged yet because...It's based on feat: Allow the build to be configured in "fail on error" mode #687 and fix: Fix confusion between shadowed variables in the AtlasStepperTests #690 , which should be merged first.The new behavior uncovered a few interesting issues in the AtlasStepperTests that must be resolved before CI is green again.=> Fixed by fix: Fix confusion between shadowed variables in the AtlasStepperTests #690