Skip to content

upkeep/4286 : add crate-ci/typos#4311

Open
Sidsector9 wants to merge 3 commits into
developfrom
upkeep/4286
Open

upkeep/4286 : add crate-ci/typos#4311
Sidsector9 wants to merge 3 commits into
developfrom
upkeep/4286

Conversation

@Sidsector9

@Sidsector9 Sidsector9 commented Apr 29, 2026

Copy link
Copy Markdown
Member

Description of the Change

Adds crate-ci/typos as a CI check on pull requests to develop. Includes a .typos.toml with project-specific casing rules (WordPress, Elasticsearch, GitHub), corrections for typos previously fixed in #4248, and Automattic as an allowlisted identifier.

Closes #4286

Log: https://github.com/10up/ElasticPress/actions/runs/25127129416/job/73643095680

Screenshot 2026-04-30 at 12 09 35 AM

How to test the Change

Changelog Entry

Developer - Spell-checking with crate-ci/typos in CI

Credits

Props @szepeviktor @felipeelia @Sidsector9

Checklist:

@Sidsector9 Sidsector9 self-assigned this Apr 29, 2026
Comment thread .github/workflows/typos.yml Fixed
@felipeelia

Copy link
Copy Markdown
Member

@Sidsector9 going through #4248, there are two strings we need to ignore: avaiableServices (a compatibility issue with an external service) and beautful (an intentional typo). I'm not seeing those being allowed here (nor being flagged by the new workflow. Can you please check? (It is also flagging three strings that we should check.)

Also, can you name the workflow "Spell Check" and the job "Typos"? Thanks!

Comment thread .typos.toml
Comment on lines +31 to +39
# Typos previously found in this codebase
"utilitary" = "utility"
"heirarchal" = "hierarchical"
"mimcs" = "mimics"
"santitize" = "sanitize"
"warnngs" = "warnings"
"versino" = "version"
"degister" = "deregister"
"quries" = "queries"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please report these in crate-ci/typos#1573

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.

Check for typos using crate-ci/typos

4 participants