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
Codecov coverage reports #6193
Codecov coverage reports #6193
Conversation
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## master #6193 +/- ##
=========================================
Coverage ? 97.37%
=========================================
Files ? 1115
Lines ? 95583
Branches ? 0
=========================================
Hits ? 93070
Misses ? 2513
Partials ? 0 ☔ View full report in Codecov by Sentry. |
Please rebase so this PR contains only codecov-related commits on top of the quantumlib/Cirq master. One way of doing the rebase is
|
I will, thank you very much. I'm learning a lot! |
2a1d156
to
2d7072b
Compare
.github/workflows/ci.yml
Outdated
env: | ||
# Admin required to access repo settings > secrets and variable > actions | ||
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} |
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.
I think we can take this out, the token is not needed for public repositories - https://github.com/codecov/codecov-action#usage.
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.
@dstrain115 has installed the Codecov App and we got our first coverage report out at
https://app.codecov.io/github/quantumlib/Cirq/commit/1fd59723ba339b9281d150d31825c3791e4f0fdc/tree 🎉
LGTM after removing the unused token settings.
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!
Testing Codecov coverage reports on a PR as per #6063 (comment)