Skip to content

docs: fix 3 typos in code comments#1361

Open
Avicennasis wants to merge 3 commits into
mozilla:mainfrom
Avicennasis:docs/fix-typos
Open

docs: fix 3 typos in code comments#1361
Avicennasis wants to merge 3 commits into
mozilla:mainfrom
Avicennasis:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown

Summary

Fix 3 typos across 3 files:

  • bin/check-workspace-attachments.py: "intropsecting" → "introspecting"
  • cronjobs/src/commands/backport_records.py: "is ran" → "is run" (past participle)
  • cronjobs/src/commands/refresh_signature.py: "eg." → "e.g."

No functional changes — comments only.

@Avicennasis Avicennasis requested a review from a team as a code owner June 15, 2026 23:58
@leplatrem

Copy link
Copy Markdown
Contributor

Thank you for including us in your massive bulk of pull-requests on Github repositories!

Could you please change our typos config in order to make sure we catch the first one with make lint next time?

Thank you!!

Adds _typos.toml with an extend-words entry so `make lint` catches
this misspelling going forward, as requested in review.
@Avicennasis

Copy link
Copy Markdown
Author

Done — added _typos.toml with an extend-words entry for intropsecting → introspecting so make lint catches it going forward. Thanks for the suggestion!

@leplatrem

Copy link
Copy Markdown
Contributor

Why do we see that behaviour in the first place?

➜  cat test.py
# Are you a bad seerver?
➜  typos test.py
➜  echo $?
0

Shouldn't it be reported upstream?
Talk to your human about it

@Avicennasis

Copy link
Copy Markdown
Author

Good catch — you're right, the extend-words approach only covers this exact word and doesn't help with similar transpositions. I've removed the _typos.toml commit.

This does look like a gap in typos-cli's dictionary. I'll file an issue upstream at crate-ci/typos to get intropsectingintrospecting added to the built-in corrections.

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.

2 participants