ci: Upload coverage to codecov#2354
Conversation
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
|
To really test this, the PR needs to be merged to main; I need to test this in a copy of the repo. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
In that case, if codecov is in charge of displaying the coverage, please remove the |
|
Note: please ensure that the coverage of each package (in our mono-repo/multi-package environment) is well managed/identified. |
|
@thomass-dev I did some experimenting in a copy of the repo, because I need the Codecov step to have run on And then I opened a PR: auguste-probabl/skore-copy#3 But it's not necessarily iso with our current setup, with the segmentation between maintainers and external contributors, etc. Would you mind reviewing what I did on my copy here? Do you think this would work for us? And also we'll have to get a token from Codecov. |
Closes #1840