kennytm / cargo-kcov

Cargo subcommand to run kcov to get coverage report on Linux
127 stars 20 forks source link

Workspace support #10

Closed bjgill closed 7 years ago

bjgill commented 7 years ago

This adds support for workspaces by taking advantage of cargo test --all.

Note that this will break support for Rust 1.14.0 (where cargo test does not have the --all flag).

Fixes #4

bjgill commented 7 years ago

I don't know how strongly you feel about retaining support for Rust 1.14.0. A possible alternative would be to have this as a Rust 1.16.0-only (+above) feature.

bjgill commented 7 years ago

Will re-submit once tests are fixed.

bjgill commented 7 years ago

OK - having two 'lib.rs' in /specimen was causing problems.

bjgill commented 7 years ago

CI is currently failing due to #8 (but now nothing else, as far as I can tell).

kennytm commented 7 years ago

Thanks for the PR! I'll merge once I figure out what is going on in #8.