Skip to content

fix: CI prints clippy version and clippy fix#7895

Merged
erickguan merged 5 commits into
mainfrom
pr-7892-clippy
Jul 10, 2026
Merged

fix: CI prints clippy version and clippy fix#7895
erickguan merged 5 commits into
mainfrom
pr-7892-clippy

Conversation

@erickguan

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #.

Rationale for this change

During dependabot run, CI prints new errors. The error is:

clippy::useless_borrows_in_formatting

I suspect it came from that ubuntu image updated clippy stable.

What changes are included in this PR?

  • print clippy version for debugging
  • fix CI errors

Are there any user-facing changes?

AI Usage Statement

GPT-5.5

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Jul 10, 2026
@erickguan erickguan requested review from dentiny and suyanhanx July 10, 2026 02:23

@dentiny dentiny left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

assume CI's happy

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 10, 2026
@erickguan

erickguan commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

hf test failure due to test source problem. So merge.

@erickguan erickguan merged commit 7b97943 into main Jul 10, 2026
190 of 191 checks passed
@erickguan erickguan deleted the pr-7892-clippy branch July 10, 2026 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants