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
Only annotate lines that the PR modifies #306
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @Floby 👋,
Thank you for this amazing PR! Just have a few notes on this.
There seems to be an issue with the package lock file as well. What would you like me to do with it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect 👍
@Floby, the package-lock file looks good to me, what is the issue? Also, it seems like the |
Any way for me to help push this forward? |
Thank you for contribution! @all-contributors please add @Floby for code |
I've put up a pull request to add @Floby! 🎉 |
hello, sorry i've been on holiday and trying to detox ^^ |
This PR adds the option
annotations = 'changed-coverage'
When active, the action will filter any coverage annotations to only include lines that the PR actually modifies.
This is useful when inheriting a poorly covered codebase, but trying not to worsen the coverage, because otherwise annotations are ignored after the 50th.
See it in action at https://github.com/Floby/test-coverage-action/pull/1/files#diff-a2a171449d862fe29692ce031981047d7ab755ae7f84c707aef80701b3ea0c80