-
Notifications
You must be signed in to change notification settings - Fork 19
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 the skip option #8
Conversation
@per1234 would you mind taking a look at my first new bats test (now it's working)? |
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.
Looks good. Thanks!
CIs are failing |
I think this is because @per1234 was testing for buggy behaviour. I expect we should fix the other things he found, then ship the next release then this will come good... |
For now...
I think this should be merge-able now @larsoner . |
Having debugged it (grumble @per1234 😜 ) it's because our behaviour for hidden files in directories has changed. |
This is sufficient to dog food:
https://github.com/codespell-project/codespell/blob/ac9e18a3ecd7021d2dbbea40f4c0a30a5d5d449d/.github/workflows/codespell.yml#L13