Skip to content

👷 Add pre-commit to check typos#15482

Merged
tiangolo merged 2 commits into
masterfrom
typos
May 5, 2026
Merged

👷 Add pre-commit to check typos#15482
tiangolo merged 2 commits into
masterfrom
typos

Conversation

@tiangolo
Copy link
Copy Markdown
Member

@tiangolo tiangolo commented May 4, 2026

👷 Add pre-commit to check typos

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 4, 2026

Merging this PR will not alter performance

✅ 20 untouched benchmarks


Comparing typos (058db6b) with master (f8cbeab)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (3efd86c) during the generation of this report, so f8cbeab was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

📝 Docs preview

Last commit e915381 at: https://a09c3381.fastapitiangolo.pages.dev

Comment thread .pre-commit-config.yaml Outdated
- id: trailing-whitespace

- repo: https://github.com/crate-ci/typos
rev: v1.46.0
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.

Suggested change
rev: v1.46.0
rev: bbaefadf97b0ec5fdc942684b647f1a6ab250274 # v1.46.0

@tiangolo tiangolo enabled auto-merge (squash) May 5, 2026 09:38
@tiangolo tiangolo merged commit fb74293 into master May 5, 2026
47 of 48 checks passed
@tiangolo tiangolo deleted the typos branch May 5, 2026 09:44
@antoine-mle
Copy link
Copy Markdown

Hi @tiangolo, just noticed this new pre-commit hook, would you then recommend typos over codespell?
Many thanks in advance for your reply!

@YuriiMotov
Copy link
Copy Markdown
Member

Hi @tiangolo, just noticed this new pre-commit hook, would you then recommend typos over codespell? Many thanks in advance for your reply!

I'm not tiangolo, but I just re-ran the workflow with codespell on SQLModel repo and it found one additional mistake (automatized -> automated): fastapi/sqlmodel#588
(release-notes is ignored by current config of typos)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants