search

AcademySoftwareFoundation / rez

An integrated package configuration, build and deployment system for software
https://rez.readthedocs.io
Apache License 2.0
932 stars 329 forks source link

Upload test coverage to Codecov #1671

Closed JeanChristopheMorinPerso closed 6 months ago

JeanChristopheMorinPerso commented 6 months ago

Fixes #1653

I initially wanted to use SonarCloud because it's already set tup, but we can't upload coverage for PRs coming from forks. That's because SonarCloud really wants an API token and GH Actions secrets are not available on PRs coming from forks.

So I instead use codecov. It's a service I use and is highly customizable and uploading coverage on fork PRs works without tokens.

This PR also enhances the development experience a bit. rez-selftest used to cd into the tests folder, which means that coverage result would be stored in the tests folder. This makes it very hard to find it, particularly when using the install script. pytest would just say "stored in htmlcov dir"... The new behavior is to not change the current working directory.

codecov[bot] commented 6 months ago

Welcome to Codecov :tada:

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered :open_umbrella: