Add github action for llvm-cov coverage#2361
Conversation
The action reports the generated report to coveralls.
|
Neat! Looks like it's working. This PR's report appears here https://coveralls.io/github/rust-bitcoin/rust-bitcoin |
|
Do you know if there's a way we can signal to coveralls to ignore the |
Seems like we can: https://github.com/taiki-e/cargo-llvm-cov#exclude-file-from-coverage Here is the change that makes it work: pool2win@47b476e Here is a coverage report without the fuzz directory: https://coveralls.io/github/pool2win/rust-bitcoin?branch=ignore-fuzz-dir-for-coveralls I can do a separate PR for the patch or push the commit to this PR - whatever you prefer. |
|
I think just updating the PR is simplest. ACK once the change is in. |
Updated the PR to ignore the |
There has been discussion around generating coverage reports for tests. See #1853 and #2353.
This PR adds an action using llvm-cov and coveralls.
The action generates lcov coverage report using llvm-cov and uploads the generated report to coveralls. You can see sample reports for my fork here: https://coveralls.io/github/pool2win/rust-bitcoin
I am using the cargo-llvm-cov wrapper around llvm-cov.
I also use the coveralls official github action to push the coverage report to coveralls. It removes the need to deal with repository secrets etc, making the action easy to run for all contributors on their forks.
We can move this action later to rust.yml if we need to.