search

rust-lang / rust-clippy

A bunch of lints to catch common mistakes and improve your Rust code. Book: https://doc.rust-lang.org/clippy/
https://rust-lang.github.io/rust-clippy/
Other
11.34k stars 1.53k forks source link

Try using coverage tracking for Clippy #3986

Open phansch opened 5 years ago

phansch commented 5 years ago

It would be nice if we were able to gather test coverage information for Clippy.

We have an old script written by @mcarton that could be used to run kcov on Clippy. It seems like this script doesn't work currently for various reasons.

While making coverage tracking work again it might be worth to investigate tarpaulin, as well.

@felix91gr mentioned on Discord that they would like to work on coverage tracking once #3963 is done.

mati865 commented 5 years ago

There is no accurate coverage available for Rust right now. https://github.com/rust-lang/rust/issues/34701 is promising but probably years from being done.

ThibsG commented 3 years ago

Recently, improvements have been done with the use of llvm-cov (see https://github.com/rust-lang/rust/issues/34701).

Zulip thread to see rendering work done by @flip1995.

To improve this rendering, one leftover question is how to exclude dependencies from the results (e.g. exclude results from .cargo and .rustup paths): maybe building deps without -Zinstrument-coverage?