Use the CodeCov GitHub Action instead of the Bash Uploader#9204
Use the CodeCov GitHub Action instead of the Bash Uploader#9204mPandaRed wants to merge 6 commits intopytest-dev:mainfrom
Conversation
|
The upload script also transforms the binary coverage data to a coverage xml report That command would have to be executed |
Is this coverage xml report also uploaded to CodeCov? |
bluetech
left a comment
bluetech
left a comment
There was a problem hiding this comment.
Thanks for working on this @mPandaRed. There's still some work needed; when it works, you should see the coverage reported for the PR.
| run: bash scripts/upload-coverage.sh -F GHA,${{ runner.os }} | ||
| uses: codecov/codecov-action@v2 | ||
| with: | ||
| files: ./.coverage,./coverage.xml |
There was a problem hiding this comment.
I think only the XML is relevant here, so you should remove the first one.
Also, you need to generate coverage.xml, it is not generated automatically. See here.
Finally, if you decide to no longer use the scripts/upload-coverage.sh script, you should remove it from git.
|
The fix has been merged in #9252. Thanks anyway! |
4 participants
Closes #9202
Using the new GitHub Action for CodeCov to upload the code coverage report instead of the bash uploader.
https://github.com/marketplace/actions/codecov