josibake / rust-bitcoin / 7601619864

84%

push

github

Merge rust-bitcoin/rust-bitcoin#2361: Add github action for llvm-cov coverage

<a class=hub.com/josibake/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/josibake/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/josibake/rust-bitcoin/commit/47b476ec5bdc2f4d9a9fd28f30a7152e1bf45277">47b476ec5<a href="https://github.com/josibake/rust-bitcoin/commit/782c2d17bd773ad12ff6035c7f55cbc9a45385e1"> Ignore fuzz dir from coverage reports (pool2win)
<a class="double-link" href="https://github.com/josibake/rust-bitcoin/commit/50ff68550cfec9340c77b99620874139dae27c2e">50ff68550</a> Add github action for llvm-cov coverage (pool2win)

Pull request description:

  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](https://github.com/taiki-e/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.

ACKs for top commit:
  apoelstra:
    ACK 47b476ec5bdc2f4d9a9fd28f30a7152e1bf45277
  Kixunil:
    ACK 47b476ec5bdc2f4d9a9fd28f30a7152e1bf45277
  tcharding:
    ACK 47b476ec5bdc2f4d9a9fd28f30a7152e1bf45277

Tree-SHA512: 7a7b8caf98fa4386d95f2f68588a48b4af2

19212 of 22810 relevant lines covered (84.23%)

16940.0 hits per line