Skip to content

Commit

Permalink
doc: Add a note about branch coverage in rust
Browse files Browse the repository at this point in the history
Specifically I added reference to rust issue 79649,
rust-lang/rust#79649 and a couple comments
I added.
  • Loading branch information
winksaville committed Aug 15, 2022
1 parent a17afdd commit 1b54b9c
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@
[![codecov](https://codecov.io/gh/winksaville/exper-code-coverage/branch/main/graph/badge.svg?token=cowZtK1KK1)](https://codecov.io/gh/winksaville/exper-code-coverage)

Experiment with code coverage and see how well [LLVM source-based code coverage](https://rustc-dev-guide.rust-lang.org/llvm-coverage-instrumentation.html) works.
> Note: I believe the library with `if_short_circuit_and()` and `short_circuit_and()`
> shows that `branch coverage` as tracked in [rust issue #79649](https://github.com/rust-lang/rust/issues/79649).
> I've provided a couple comments to that effect [here](https://github.com/rust-lang/rust/issues/79649#issuecomment-1214483770)
> and [here](https://github.com/rust-lang/rust/issues/79649#issuecomment-1215764155).
Short answer seems to be that llvm-cov is better than tarpaulin as I
never see 100% for trapaulin. But llvm-cov is not perfect. I've
Expand Down

0 comments on commit 1b54b9c

Please sign in to comment.