ci: configure codespell in pre-commit#1478
Conversation
|
@IndexSeek thanks for the PR. could you fix the lint issue by running |
You're welcome! Good catch, sorry, forgot to lint! I added the header to the file and resolved a few codespelling violations to get pre-commit to pass. |
hussein-awala
left a comment
There was a problem hiding this comment.
This looks perfect!
Nit: IMHO it would be better to move the ignore list to pyproject.toml if it’s possible, but let’s wait for committers’ reviews before moving it
Fokko
left a comment
There was a problem hiding this comment.
This looks great, thanks @IndexSeek for raising this PR, and thanks @kevinjqliu and @hussein-awala for reviewing this 🚀
Following up with #1471 to detect spelling issues with pre-commit.
I went with adding a
.codespellrcfile to store theignore-words-list.I think it's possible to configure this in thepyproject.tomlas well. We could probably swap that out if desired.