MNT Disable test coverage report in ci #18839
Conversation
|
For both issues, do we want to use |
|
The XFAIL summary is probably useless. The SKIPPED summary is probably useful to look at from time to time to check whether some tests are not always skipped on the CI for the wrong reason. But it would actually be better to consolidate the list of skipped tests and check that a given test is not skipped on all the CI workers as we do in threadpoolctl: https://github.com/joblib/threadpoolctl/blob/master/continuous_integration/check_no_test_skipped.py |
|
Let's go with the original solution. Newer version of pytest + python 3.6 have had weird issues. |
I was wondering if they were fixed. I will revert. |
529daca to
f1cd72f
Compare
|
Forced pushed the removal of the last 2 commits that tried to use |
I find it really annoying to have to scroll up several pages in the CI logs to find the interesting pytest error messages. The coverage reports are better consolidated by codecov anyways. So let's disable the text report which is tedious to parse anyway.
Related to #18833.