-
Notifications
You must be signed in to change notification settings - Fork 333
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
Add typos
as a pre-commit hook
#2700
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2700 +/- ##
=======================================
Coverage 95.25% 95.25%
=======================================
Files 104 104
Lines 9435 9435
Branches 2159 2159
=======================================
Hits 8987 8987
Misses 272 272
Partials 176 176 ☔ View full report in Codecov by Sentry. |
_typos.toml
Outdated
@@ -0,0 +1,19 @@ | |||
[default] | |||
extend-ignore-identifiers-re = [ |
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.
Do you want these two sections?
|
||
[default.extend-words] | ||
aas = "aas" | ||
arange = "arange" |
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.
This was such an annoying thing to have to add to our config as well.
This PR tries out typos as a pre-commit hook as a possible addition to or replacement for codespell. It managed to find a couple more typos. Since it's written in Rust, it's probably quicker (though I didn't time it).
I'll tentatively add this as a hook without removing codespell, but we can consider removing one of them later if it gets annoying to have to deal with both.
Thanks to @nabobalis for suggesting this!