Skip to content

Library: enforce clippy deref lints in CI - #161017

Open
ChrisDenton wants to merge 6 commits into
rust-lang:mainfrom
ChrisDenton:clippy2
Open

Library: enforce clippy deref lints in CI#161017
ChrisDenton wants to merge 6 commits into
rust-lang:mainfrom
ChrisDenton:clippy2

Conversation

@ChrisDenton

Copy link
Copy Markdown
Member

Best reviewed commit by commit.

The output is from clippy --fix but I've reviewed the changes and am feeling confident there are no behaviour changes. Still, a second pair of eyes would be useful.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 13, 2026
@rustbot

rustbot commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

r? @JohnTitor

rustbot has assigned @JohnTitor.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: libs
  • libs expanded to 12 candidates
  • Random selection from JohnTitor, Mark-Simulacrum, clarfonthey, nia-e

Comment thread src/bootstrap/src/core/build_steps/clippy.rs Outdated
@clarfonthey

clarfonthey commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

For reference, the included lints are:

  • needless_borrow
  • op_ref
  • borrow_deref_ref
  • explicit_auto_deref

@clarfonthey

Copy link
Copy Markdown
Contributor

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.

@ChrisDenton

ChrisDenton commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

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-*

rust-bors Bot pushed a commit that referenced this pull request Aug 13, 2026
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-*
@rust-bors

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@nia-e

nia-e commented Aug 13, 2026

Copy link
Copy Markdown
Member

i handled the last one, so i'm happy lightening the review load ^^

r? me

@rustbot rustbot assigned nia-e and unassigned JohnTitor Aug 13, 2026
@rustbot rustbot added the O-unix Operating system: Unix-like label Aug 13, 2026
@rust-bors rust-bors Bot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 13, 2026
@rust-bors

rust-bors Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

💔 Test for b0329c8 failed: CI. Failed job:

@ChrisDenton

Copy link
Copy Markdown
Member Author

Hm, the logs just aren't there. Weird. Let me try again

@bors try jobs=x86_64-msvc-1

@rust-bors

rust-bors Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

⌛ Trying commit 7f74e59 with merge d8e77f5

To cancel the try build, run the command @bors try cancel.

Workflow: https://github.com/rust-lang/rust/actions/runs/31695315900

rust-bors Bot pushed a commit that referenced this pull request Aug 13, 2026
Library: enforce clippy deref lints in CI


try-job: x86_64-msvc-1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-unix Operating system: Unix-like S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants