-
Notifications
You must be signed in to change notification settings - Fork 157
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
ci: Improve physmon report comment #1648
Conversation
📊 Physics performance monitoring for 8a55d6aFull report VertexingCKFAmbiguity resolutionTruth tracking |
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.
Very nice! just a few very minor comments, feel free to ignore them
Codecov Report
@@ Coverage Diff @@
## main #1648 +/- ##
=======================================
Coverage 48.49% 48.49%
=======================================
Files 384 384
Lines 21059 21059
Branches 9694 9694
=======================================
Hits 10213 10213
Misses 4122 4122
Partials 6724 6724 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
With these changes, if the physmon job didn't complete successfully, the report comment will look something like this: