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.

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

Ignore cargo registry from docker image #104

Closed
nim65s opened this issue Nov 15, 2021 · 2 comments · Fixed by #105
Closed

Ignore cargo registry from docker image #104

nim65s opened this issue Nov 15, 2021 · 2 comments · Fixed by #105
Labels
C-enhancement Category: A new feature or an improvement for an existing one

Comments

@nim65s
Copy link

nim65s commented Nov 15, 2021

Hi,

When using this project inside the official rust docker image, the coverage report include stuff from /usr/local/cargo/registry/src/.

I'm not sure how to read the exclude regex in

r"(^|{0})(rustc{0}[0-9a-f]+|.cargo{0}(registry|git)|.rustup{0}toolchains|tests|examples|benches|target{0}llvm-cov-target){0}",

so I don't know if it's just me doing something wrong or not.

Is there a way to exclude everything from this path ?

@taiki-e taiki-e added the C-enhancement Category: A new feature or an improvement for an existing one label Nov 15, 2021
@bors bors bot closed this as completed in f24528a Nov 15, 2021
@bors bors bot closed this as completed in #105 Nov 15, 2021
@taiki-e
Copy link
Owner

taiki-e commented Nov 15, 2021

Fixed in v0.1.12. (#105)

@nim65s
Copy link
Author

nim65s commented Nov 15, 2021

Perfect, thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: A new feature or an improvement for an existing one
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants