-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore(CI): Report coverage as a PR comment #787
chore(CI): Report coverage as a PR comment #787
Conversation
44890fd
to
43e5880
Compare
43e5880
to
3830c25
Compare
Remind me, why does the coverage report include the |
You right, this makes no sense. Hence I've added the tests folder to the omit rule of coverage. |
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.
Looks good to me!
Please don't merge, my review is not finished. |
Co-authored-by: Thomas S. <[email protected]>
skip-checks:true
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.
Strong work, thanks.
A bot will now push(and edit) a single comment with coverage information.
To achieve that this PR: