Skip to content

Add clippy linting to all packages - #7116

Draft
matthewhughes934 wants to merge 1 commit into
rust-lang:mainfrom
matthewhughes934:add-clippy-repeated-config
Draft

Add clippy linting to all packages#7116
matthewhughes934 wants to merge 1 commit into
rust-lang:mainfrom
matthewhughes934:add-clippy-repeated-config

Conversation

@matthewhughes934

@matthewhughes934 matthewhughes934 commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Duplicate the same config across each package and add a script that verifies these configs are all equal, and add a workflow so that clippy is run across all packages in PRs.

Compare with: #7116 this change has

  • Con: Adds more things (the extra script+duplicate configs)
  • Pro: let's developers just invoke plain cargo clippy in any package, which works nicely with common workflows like IDE integrations
  • I did not use an LLM to create a change in this PR.
  • I used an LLM to create a change in this PR, and I have explained below how it was used.

@rustbot rustbot added A-CI Area: CI S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author. labels Sep 12, 2026
@matthewhughes934
matthewhughes934 force-pushed the add-clippy-repeated-config branch 2 times, most recently from 50d2f06 to 1f797ab Compare September 12, 2026 15:18
Duplicate the same config across each package and add a script that
verifies these configs are all equal, and add a workflow so that
`clippy` is run across all packages in PRs.
@matthewhughes934
matthewhughes934 force-pushed the add-clippy-repeated-config branch from 1f797ab to b8e813a Compare September 12, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: CI S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants