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
Spellcheck on CI #6890
Spellcheck on CI #6890
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.
LGTM
@Mouvedia Thanks for the review! |
Any specific reason for choosing |
I didn't have a strong opinion, but didn't want to add extra npm dependencies. |
It sounds like that you prefer to relying on CI checks only instead of checking locally before committing? |
Yes. |
None.
This PR introduces spellcheck on CI by using
codespell
(written in Python).See:
I automatically fixed some misspells via the following command:
But currently, there are the following false positives. It seems that fixing these doesn't make sense to me.