-
Notifications
You must be signed in to change notification settings - Fork 22
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. <thomas@probabl.ai>
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: - uses a market place action to generate the comment - to optimize action's duration coverage as been disable in the matrixed testing step - to optimize action's and as frontend serving test was not pertinent it has been removed - coverage is only run once in another task (linux | python 3.12) - frontend artifact related stuff have been removed --------- Co-authored-by: Thomas S. <thomas@probabl.ai>
A bot will now push(and edit) a single comment with coverage information.

To achieve that this PR: