Skip to content

Fix clippy lints in librustdoc #144178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 20, 2025
Merged

Conversation

GuillaumeGomez
Copy link
Member

No description provided.

@rustbot
Copy link
Collaborator

rustbot commented Jul 19, 2025

r? @notriddle

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

@rustbot rustbot added A-rustdoc-json Area: Rustdoc JSON backend S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Jul 19, 2025
@fmease
Copy link
Member

fmease commented Jul 19, 2025

Spellcheck error is "spurious" or rather it affects all PRs rn and it + infra needs to be fixed elsewhere. And since auto builds / homu likely doesn't care about the spellcheck workflow, I'll put it in the queue despite the failure.

Thanks! @bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 19, 2025

📌 Commit c75edc9 has been approved by fmease

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 19, 2025
@fmease
Copy link
Member

fmease commented Jul 19, 2025

Spellcheck error is "spurious" or rather it affects all PRs rn and it + infra needs to be fixed elsewhere.

#t-infra > Spellcheck workflow now fails on all PRs (tree bad?) @ 💬

bors added a commit that referenced this pull request Jul 19, 2025
Rollup of 5 pull requests

Successful merges:

 - #143960 (Tidy cleanup 2)
 - #144149 (docs: update link to RISC-V and Xtensa installation guide)
 - #144152 (Rename `optional-mingw-check-1` to `optional-pr-check-1`)
 - #144178 (Fix clippy lints in librustdoc)
 - #144180 (fix load-bearing typo)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Jul 19, 2025
Rollup of 5 pull requests

Successful merges:

 - #143960 (Tidy cleanup 2)
 - #144149 (docs: update link to RISC-V and Xtensa installation guide)
 - #144152 (Rename `optional-mingw-check-1` to `optional-pr-check-1`)
 - #144178 (Fix clippy lints in librustdoc)
 - #144180 (fix load-bearing typo)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit a6b625c into rust-lang:master Jul 20, 2025
10 of 11 checks passed
@rustbot rustbot added this to the 1.90.0 milestone Jul 20, 2025
rust-timer added a commit that referenced this pull request Jul 20, 2025
Rollup merge of #144178 - GuillaumeGomez:fix-clippy-lints, r=fmease

Fix clippy lints in librustdoc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rustdoc-json Area: Rustdoc JSON backend S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants