Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Try using coverage tracking for Clippy #3986

Open
phansch opened this issue Apr 18, 2019 · 2 comments
Open

Try using coverage tracking for Clippy #3986

phansch opened this issue Apr 18, 2019 · 2 comments
Labels
C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages

Comments

@phansch
Copy link
Member

phansch commented Apr 18, 2019 •
edited
Loading

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.
@phansch phansch added the C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages label Apr 18, 2019
@phansch phansch changed the title Try enabling coverage tracking for Clippy (again) Try using coverage tracking for Clippy Apr 18, 2019
@mati865
Copy link
Contributor

mati865 commented Apr 18, 2019

There is no accurate coverage available for Rust right now. rust-lang/rust#34701 is promising but probably years from being done.

@ThibsG
Copy link
Contributor

ThibsG commented Nov 13, 2020

Recently, improvements have been done with the use of llvm-cov (see rust-lang/rust#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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mati865 @phansch @ThibsG