-
Notifications
You must be signed in to change notification settings - Fork 2
Update coverage report deployment in PR #33
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #33 +/- ##
==========================================
+ Coverage 82.38% 82.45% +0.06%
==========================================
Files 16 16
Lines 1272 1271 -1
==========================================
Hits 1048 1048
+ Misses 224 223 -1 ☔ View full report in Codecov by Sentry. |
Weighted Code CoverageProject metrics:
|
9ff0993 to
ceb6f6d
Compare
ceb6f6d to
e7b20a8
Compare
Luni-4
left a comment
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.
Sorry for the late reply! This PR is great! Just simple changes and we can land it
e7b20a8 to
b8ef4b2
Compare
b8ef4b2 to
387f3d4
Compare
Luni-4
left a comment
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.
Thanks a lot!
This PR updates the workflow to include the deployment of the
HTMLcoverage report usingsmokeshow. After the site is created, the URL is extracted and included in the PR comment.