search

gendx / lzma-rs

An LZMA decoder written in pure Rust
MIT License
127 stars 27 forks source link

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

Closed gendx closed 2 years ago

gendx commented 2 years ago

Pull Request Overview

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

Fixes https://github.com/gendx/lzma-rs/issues/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 commented 2 years ago

Codecov Report

:exclamation: 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 ```diff @@ 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](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

gendx commented 2 years ago

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