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

force full re-clippy #2498

Closed
davemilter opened this issue Mar 3, 2018 · 6 comments
Closed

force full re-clippy #2498

davemilter opened this issue Mar 3, 2018 · 6 comments

Comments

@davemilter
Copy link

I want run clippy on my CI server and get every time all warnings,
not only in changed projects.
Also I want to cache "target" directory between run of cargo build,
because of compilation too slow.

But I don't see how it possible with current clippy:

$ cargo +nightly clippy
warning:
warning:
warning:
$ cargo +nightly clippy
$ lib: foo
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
test: test_foo
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs

as you see the second run of clippy gives nothing,
And there is no option like --force, as workaround I use find . -name "*.rs" -exec touch {} \;,
it is better then remove target or cargo clean, but still cause also rebuild of crate in the next cargo +nigtly test

@birkenfeld
Copy link
Contributor

Yep, this is annoying - the purpose of running a linter is to emit lints on every run :)

@lnicola
Copy link
Member

lnicola commented Mar 28, 2018

Did #2582 fix this?

@oli-obk
Copy link
Contributor

oli-obk commented Mar 28, 2018

My local tests say yes. Reopen a new issue if it persists on your system

@dessalines
Copy link

This is still an issue.

@simnalamburt
Copy link

Me too. Is there any explicit options to force re-run?

@flip1995
Copy link
Member

tracked in #4612

on nightly you can use cargo clippy -Zunstable-options

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants