-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix spelling errors #9844
Fix spelling errors #9844
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's hope nobody depends on the misspelled error message :D
I wonder if we can have something like this as a CI check / GitHub review bot thingy? |
That’s definitely possible to some extent. I found that looking for common digraphs but with the letters transposed led to finding typos in the list much quicker but I do fear that automating it would produce lots of false positives. |
Ah, I didn't realize you filtered by hand. Nevermind me and thank you! :) |
Thank you @camreeves. 🙇 Those were a lot of tpos! |
Similar to fixes done in #9844.
* Fix spelling errors (3) Similar to fixes done in #9844 and #9854. This commit completes the review of all files in the repository. * Fix spelling errors (3.1) * Undo an incorrect word replacement. Thanks @makenowjust for pointing it out in #9854.
A friend created a long list (3500+ entries) with cspell for the codebase and I've manually corrected a bunch.
A thank you to @caspiano for the initial list.