Upload coverage to codecov, even if some tests fail

[dstansby]

In sunpy we often have remote data tests failing because of remote data provider issues. In this case we often merge PRs with failing tests, but want the code coverage to be measured on the remote data runs that have some failing tests. It would therefore be good to uplaod code coverage even if some tests fail.

[Cadair]

It looks like this is something that you can configure directly with codecov: https://docs.codecov.com/docs/common-recipe-list#see-coverage-on-ci-test-failures

[ConorMacBride]

The codecov config doesn't control whether results are uploaded, just how the report is presented. We would need to add an "if: always" to the upload step, but that may not be wanted in most cases.

[Cadair]

Ah, I see.