Add codespell pre-commit-hook #9781
Conversation
headtr1ck
added
the
Automation
|
Looks good! |
|
Does anyone have an idea why pre-commit fails in CI? It seems to ignore the skip of whats-new |
| "PIE", # flake8-pie | ||
| "TID", # flake8-tidy-imports (absolute imports) | ||
| "PGH", # pygrep-hooks | ||
| "F", # Pyflakes |
There was a problem hiding this comment.
Doesn't black usually do 2 whitespaces for comments?
There was a problem hiding this comment.
Oh, good question.
I shouldn't have even formatted this even, but pre-commit did that ;)
There was a problem hiding this comment.
which hook was that? As far as I can tell, we don't have anything that would format TOML (yet, we can totally change that).
There was a problem hiding this comment.
Oh, might have been format on save in vscode.
I will have to check that.
There was a problem hiding this comment.
we don't have anything that would format TOML (yet, we can totally change that).
Stumbled across this and thought I'd make #9840 :)
Given the large codebase there were surprisingly few misspellings.
But even better to add a pre-commit hook for that :)