Add codespell support (config, workflow to detect/not fix) and make it fix some typos #370
Conversation
…nteractively
=== Do not change lines below ===
{
"chain": [],
"cmd": "codespell -w -i 3 -C 4 ./tests/test_io/test_openephys.py",
"exit": 0,
"extra_inputs": [],
"inputs": [],
"outputs": [],
"pwd": "."
}
^^^ Do not change lines above ^^^
…os automagically
=== Do not change lines below ===
{
"chain": [],
"cmd": "codespell -w",
"exit": 0,
"extra_inputs": [],
"inputs": [],
"outputs": [],
"pwd": "."
}
^^^ Do not change lines above ^^^
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #370 +/- ##
=======================================
Coverage 89.92% 89.92%
=======================================
Files 12 12
Lines 2044 2044
=======================================
Hits 1838 1838
Misses 206 206 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Would recommend, @yarikoptic introduced this to a few of our repos and it never has false positives + has caught many typos! |
|
It does a small number of false positives for us but I think it is worth it: I am +1 on this. But I am behind @samuelgarcia if he does not want more linters. I would not burn our pre-commit/linter credits on this one if we only get a few : p |
I think we have a quorum :) Let's go for it! |
4 participants
More about codespell: https://github.com/codespell-project/codespell .
I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.
CI workflow has 'permissions' set only to 'read' so also should be safe.
I excluded release notes but could be fixed too:
wanna?