Thanks to visit codestin.com
Credit goes to github.com

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

@tiangolo tiangolo commented May 4, 2026

Copy link
Copy Markdown
Member

👷 Add pre-commit to check typos

@codspeed-hq

codspeed-hq Bot commented May 4, 2026

Copy link
Copy Markdown

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

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

📝 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