-
Notifications
You must be signed in to change notification settings - Fork 262
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
Run again tests with new default checks #8106
Run again tests with new default checks #8106
Conversation
32cbc52
to
168468c
Compare
aeea845
to
58f05d9
Compare
This reverts commit 457b953.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #8106 +/- ##
===========================================
+ Coverage 79.08% 79.09% +0.01%
===========================================
Files 1695 1695
Lines 196514 196514
===========================================
+ Hits 155412 155436 +24
+ Misses 41102 41078 -24 ☔ View full report in Codecov by Sentry. |
… runner scripts" This reverts commit 2b6446e.
… test runner scripts" This reverts commit f79b5bc.
…ner scripts" This reverts commit ce65686.
…ipts" This reverts commit 5f17296.
…er script" This reverts commit 7e38518.
…er script" This reverts commit 81c73d0.
…er script" This reverts commit 0d19437.
439e84c
to
12ff897
Compare
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.
Thanks for the painstaking work to bring these all back. Merry Christmas!
12ff897
to
77705eb
Compare
This is reverting changes currently in
develop
which were applying the--no-standard-checks
across allCMakeLists.txt
andmakefile
s in theregression
folder while we were trying to stabilise the new standard check flags in #8093.This should bring all relevant flags for test execution into the appropriate
*.desc
files to make it easier to see what flags a test run actually executes with.