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
Typos fixed #2893
Typos fixed #2893
Conversation
LGTM! I also tried to automate this as part of our |
@Zac-HD thanks a lot for your help! I will finish it next week 👍 |
It took me more time than a week 😅 |
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.
Two tiny nitpicks, but looks good to me overall - thanks for fixing this up, and feel free to merge when you think it's done.
"--check-hidden", | ||
"--check-filenames", | ||
"--ignore-words=./tooling/ignore-list.txt", | ||
"--skip=__pycache__,_build,.mypy_cache,.venv,.git,tlds-alpha-by-domain.txt", |
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.
"--skip=__pycache__,_build,.mypy_cache,.venv,.git,tlds-alpha-by-domain.txt", |
These are already handled by the logic which chooses while files to format, or the comprehension in lint()
.
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.
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.
Weird! If that's how it works though, let's leave it as-is 🙂
Thanks a lot, @Zac-HD! 👍 |
I am using https://github.com/codespell-project/codespell to find typos, it works pretty well!
We can even automate this check later on:
whitelist.txt
of words that are ignored, likestrat
Full example: https://github.com/dry-python/returns/blob/master/setup.cfg#L191-L194