-
Notifications
You must be signed in to change notification settings - Fork 616
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: use codecov/codecov-action #3986
Conversation
Use codecov/codecov-action instead of the codecov CLI.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3986 +/- ##
=======================================
Coverage 85.39% 85.39%
=======================================
Files 235 235
Lines 56628 56628
=======================================
Hits 48358 48358
Misses 8270 8270 ☔ View full report in Codecov by Sentry. |
Thank you very much. |
I've removed the old settings for codecov CLI and for coveralls. @masatake Do you think it's okay? |
cda666f
to
4d16235
Compare
I tried to remove lcov, but it didn't work well. It seems that lcov is still needed even with codecov/codecov-action. |
* Remove old settings for the codecov CLI. * Remove old settings for coveralls.
4d16235
to
7be24dc
Compare
Everything reducing the maintenance cost of the files under .github/workflow is acceptable in this area! |
Thank you again. |
Use codecov/codecov-action instead of the codecov CLI.