Skip to content

fix(ci): various clippy CI follow-ups - #7124

Merged
jieyouxu merged 4 commits into
mainfrom
ci-lock
Sep 17, 2026
Merged

jieyouxu merged 4 commits into
mainfrom
ci-lock

Conversation

@jieyouxu

@jieyouxu jieyouxu commented Sep 17, 2026

Copy link
Copy Markdown
Member

Follow-ups to #7116:

  • Also run clippy on check_diff, and use the same preset for check_diff.
  • Use --locked for CI clippy related invocations.
  • Remove an unnecessary explicit rustup target addition.

  • 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-review Status: awaiting review from the assignee but also interested parties. labels Sep 17, 2026

@matthewhughes934 matthewhughes934 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦 I can't believe I missed these, nice follow-up

@jieyouxu

Copy link
Copy Markdown
Member Author

I can't believe I missed these, nice follow-up

Very easy to miss :D Don't worry about it.

@rustbot rustbot added this to the 1.100.0 milestone Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: CI release-notes Needs an associated changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants