Skip to content
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

Add coverage report action with cargo-llvm-cov & Codecov. #86

Merged
merged 2 commits into from
Oct 30, 2022
Merged

Conversation

gendx
Copy link
Owner

@gendx gendx commented Oct 27, 2022

Pull Request Overview

This pull request adds code coverage report using cargo-llvm-cov, uploaded to Codecov.

Fixes #84.

Testing Strategy

This pull request was tested by checking whether the GitHub Action works.

Supporting Documentation and References

TODO or Help Wanted

N/A

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@4989abd). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #86   +/-   ##
=========================================
  Coverage          ?   86.83%           
=========================================
  Files             ?       19           
  Lines             ?     2484           
  Branches          ?        0           
=========================================
  Hits              ?     2157           
  Misses            ?      327           
  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.
📢 Do you have feedback about the report comment? Let us know in this issue.

@gendx gendx marked this pull request as ready for review October 30, 2022 15:51
@gendx
Copy link
Owner Author

gendx commented Oct 30, 2022

Based on the analysis of the reports on Codecov, llvm-cov yields much better results than grcov.

@gendx gendx merged commit adfba36 into master Oct 30, 2022
@gendx gendx deleted the llvm-cov branch October 30, 2022 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Report code coverage
2 participants