-
Notifications
You must be signed in to change notification settings - Fork 599
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] Update coverage tool to codecov for pre-merge #885
Conversation
06f463d
to
926ad91
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #885 +/- ##
=======================================
Coverage ? 86.85%
=======================================
Files ? 165
Lines ? 6331
Branches ? 0
=======================================
Hits ? 5499
Misses ? 832
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@yunchu, thanks for your PR. Is it possible to do this without setting it up from https://app.codecov.io/gh/openvinotoolkit? |
@samet-akcay AFAIK, for the public repo, it doesn't need to register it and corresponding token is also not required. |
hm, if this is the case, it's good news! |
um... it looks like to have signing up to the codecov.io |
I guess the number of the coverage for the default branch should be reported to the codecov to show it correctly. |
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 for adding this
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!
Description
Changes
Checklist