trailofbits / dylint

Run Rust lints from dynamic libraries

Home Page:https://blog.trailofbits.com/2021/11/09/write-rust-lints-without-forking-clippy/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Is there a way to skip checking dependency crates in the same workspace?

EFanZh opened this issue · comments

commented

Clippy provides a --no-deps option to do this: rust-lang/rust-clippy#6188. Is is possible to simulate this behavior in dylint?

Dylint does not have this feature currently, but we could add it.

I want to be sure I understand the problem that it solves, though. A workspace has two crates, A and B, where A depends on B. B causes some lint to trigger. We want to lint A, and not trigger the lint/see warnings for B. Is that right?

Is having this feature time sensitive?

commented

Yes, that is right. And it is not time sensitive. I can simulate this behavior manually. But it might be reasonable for dylint to support it officially.

I'll start working on it this week.

This should be fixed by #808 in version 2.2.0 (just released).

However, it required messing with Cargo caching, which I find especially tricky to get right.

If anything about the new feature doesn't work as you expect, please don't hesitate to re-open this issue.