Closed nawtrey closed 2 years ago
It appears the code coverage is currently generating the coverage report based on a single runner's results instead of combining the reports, so that needs to be fixed.
Unrelated: Probably want to change pytest
command as well. We don't need pytest-xdist
, so we can remove that portion along with the -n 10
which is likely doing nothing but generating heat.
It would be nice if the
codecov
behavior was more similar to some of the larger OSS projects, such as scipy.Specific changes to make:
codecov
PR bot comments offpytest
is generating a report in the appropriate format socodecov
can pick it upcodecov
isn't ever going to fail the CI by setting the minimum values to 1/0 for the project/patch coverage values.