Skip to content

Add Code Coverage to PR Checks #7290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ericspod opened this issue Dec 5, 2023 · 1 comment · Fixed by #8402
Closed

Add Code Coverage to PR Checks #7290

ericspod opened this issue Dec 5, 2023 · 1 comment · Fixed by #8402
Labels

Comments

@ericspod
Copy link
Member

ericspod commented Dec 5, 2023

Is your feature request related to a problem? Please describe.
Codecov and others will add a report on the coverage metrics for PRs. This would be helpful for us when assessing the coverage of tests for new features and enforce the need to improve coverage and not let it slide.

Describe the solution you'd like
A PR comment report of the coverage delta for the proposed changes, like this.

Describe alternatives you've considered
Manual code coverage checking.

Additional context
We use Codecov now for tracking coverage but adding the PR message would help.

@chrislevn
Copy link
Contributor

chrislevn commented Mar 25, 2025

Hi @ericspod, I'm new to MONAI and would love to contribute. I have a draft PR here. Will improve it upon your feedback. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants