-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 mistakes detected by newer codespell #2683
Conversation
Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):
|
This is needed to bring @nicksavers @Arachnid @Souptacular @gcolvin anyone wants to approve/merge? |
As an author of 1193, I want to second @axic's call for merging this. We're at the end of our review period and need to get a final set of changes merged ASAP. |
Can we please disable spellcheck as a required part of the build? It's absurd that a spelling error in one EIP can break the whole build - and since spell checkers have false positives, it shouldn't be required. |
I'll chime in to say that the Also, we unfortunately seem to have missed one failure: #2691 |
* Add iam to codespell-whitelist (used as a variable name) * Allow french spelling in the whitelist * Allow ACN (australian company number) in the whitelist * Fix misspellings
* Add iam to codespell-whitelist (used as a variable name) * Allow french spelling in the whitelist * Allow ACN (australian company number) in the whitelist * Fix misspellings
* Add iam to codespell-whitelist (used as a variable name) * Allow french spelling in the whitelist * Allow ACN (australian company number) in the whitelist * Fix misspellings
Fixes #2674.