search

rust-lang / rust-clippy

A bunch of lints to catch common mistakes and improve your Rust code. Book: https://doc.rust-lang.org/clippy/
https://rust-lang.github.io/rust-clippy/
Other
10.89k stars 1.46k forks source link

Add `cargo dev setup toolchain` #12680

Closed Alexendoo closed 3 weeks ago

Alexendoo commented 3 weeks ago

Adds a cargo dev setup toolchain subcommand that creates a rustup toolchain with symlinks to the local cargo-clippy and clippy-driver. Allows you to then do cargo +clippy clippy in other projects to run the locally built Clippy

Sometimes more convenient when you're testing changes on a separate project than cding back & forth to use cargo dev lint [project]

changelog: none

rustbot commented 3 weeks ago

r? @llogiq

rustbot has assigned @llogiq. They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

llogiq commented 3 weeks ago

There's a lot to unwrap() here, pun intended. I guess for an internal tool that's ok though.

Thank you for building this.

@bors r+

bors commented 3 weeks ago

:pushpin: Commit 1fff3bef4de408bdea2dd005755770ed3a92bb3d has been approved by llogiq

It is now in the queue for this repository.

bors commented 3 weeks ago

:hourglass: Testing commit 1fff3bef4de408bdea2dd005755770ed3a92bb3d with merge 1ba78212ffbe152c4921a8e39b1bd31b8d13523c...

bors commented 3 weeks ago

:sunny: Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test Approved by: llogiq Pushing 1ba78212ffbe152c4921a8e39b1bd31b8d13523c to master...