Library: enforce clippy deref lints in CI - #161017
Conversation
|
r? @JohnTitor rustbot has assigned @JohnTitor. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
For reference, the included lints are:
|
|
Also fwiw I did review this and agree it looks like everything is fine, although figure we probably want someone who isn't as enthusiastic about enabling clippy lints everywhere to take a look and make sure we're not being overzealous. So, I'll let someone else perform that aspect of "review" rather than just merging this. |
|
Let my try a few platforms to catch any platform-specific code that hits these lints. I'm not entirely sure which ones actually use clippy in CI so I'll just try a selection: @bors try jobs=arm-android,i686-gnu-,test-various,x86_64-gnu,aarch64-apple-,x86_64-msvc-,i686-msvc-,x86_64-mingw-* |
Library: enforce clippy deref lints in CI try-job: arm-android try-job: i686-gnu-* try-job: test-various try-job: x86_64-gnu try-job: aarch64-apple-* try-job: x86_64-msvc-* try-job: i686-msvc-* try-job: x86_64-mingw-*
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
i handled the last one, so i'm happy lightening the review load ^^ r? me |
|
💔 Test for b0329c8 failed: CI. Failed job:
|
|
Hm, the logs just aren't there. Weird. Let me try again @bors try jobs=x86_64-msvc-1 |
|
⌛ Trying commit 7f74e59 with merge d8e77f5… To cancel the try build, run the command Workflow: https://github.com/rust-lang/rust/actions/runs/31695315900 |
Library: enforce clippy deref lints in CI try-job: x86_64-msvc-1
Best reviewed commit by commit.
The output is from
clippy --fixbut I've reviewed the changes and am feeling confident there are no behaviour changes. Still, a second pair of eyes would be useful.