-
Notifications
You must be signed in to change notification settings - Fork 11
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
Codespell: adding config, workflow and fixing some typos #336
Conversation
I don't see how these type fixes could result in appveyor fails like
|
I have seen these failures also in core in the metalad tests, e.g. in datalad/datalad#7308. It looks like the fix is in https://github.com/datalad/datalad-metalad/pull/333/files and landed in maint - you probably found that already and thus created the merge of maint - thx! |
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.
Nice. Thanks @yarikoptic
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #336 +/- ##
=======================================
Coverage 86.42% 86.42%
=======================================
Files 88 88
Lines 4831 4831
=======================================
Hits 4175 4175
Misses 656 656
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
heh -- codespell is still failing due to fresh? |
=== Do not change lines below === { "chain": [], "cmd": "codespell -w", "exit": 0, "extra_inputs": [], "inputs": [], "outputs": [], "pwd": "." } ^^^ Do not change lines above ^^^
7e8666f
to
11932fa
Compare
In the excitement of coming up hackathon and also since datalad/datalad-extension-template#45 has been debated for nearly half a year, decided to make at least this one a little more a better spelled one ;)