Skip to content

Conversation

@madsmtm
Copy link
Contributor

@madsmtm madsmtm commented Oct 28, 2025

By running the typos CLI tool, as well as a manual fix for a typo found in #967 (comment).

r? nnethercote

Rendered

By running the `typos` CLI tool, as well as a manual fix found in:
#967 (comment)
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 28, 2025
@nnethercote
Copy link

Nice fixes!

I wondered why these weren't detected by the CI typos check added in #134006; it's because that only checks certain directories (compiler/, library/, src/bootstrap/, src/librustdoc/). It feels to me like the typos check hasn't caused any problems, and it might be worth expanding the directories it covers. cc @klensy

@bors r+ rollup

@jieyouxu
Copy link
Member

@nnethercote I think you might've confused this repo with r-l/r, this is the rust-forge repo, where AFAIK we don't have typos setup, neither do we have bors here :D

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Thanks

@jieyouxu jieyouxu merged commit dc8bc60 into master Oct 29, 2025
1 check passed
@jieyouxu jieyouxu deleted the madsmtm/typos branch October 29, 2025 04:58
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 29, 2025
@nnethercote
Copy link

you might've confused this repo with r-l/r

Indeed I did! typos seems to be a very good tool.

@klensy
Copy link

klensy commented Oct 29, 2025

Yep, typos installed only for rust repo by me :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants