Skip to content

Add cargo alias to run clippy - #7117

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

matthewhughes934 wants to merge 1 commit into
rust-lang:mainfrom
matthewhughes934:add-clippy-alias

Conversation

@matthewhughes934

Copy link
Copy Markdown
Contributor

The alias allows us to centralise the lints we allow/deny in a single location. Also add a workflow to run this across all packages in PRs.

  • 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
@ytmimi

ytmimi commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Might want to add a section to the contributing docs that explains how to run clippy before submitting PRs.

The alias allows us to centralise the lints we allow/deny in a single
location. Also add a workflow to run this across all packages in PRs.
@matthewhughes934

Copy link
Copy Markdown
Contributor Author

Might want to add a section to the contributing docs that explains how to run clippy before submitting PRs.

👍 done

@ytmimi ytmimi mentioned this pull request Sep 15, 2026
2 tasks
@ytmimi

ytmimi commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Been thinking about this, and although I'm not thrilled about the duplication I think the fact that we can run cargo clippy and have it "just work" is ultimately better for developers so let's go with #7116.

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.

3 participants