search

codespell-project / actions-codespell

MIT License
74 stars 19 forks source link

Add the skip option #8

Closed peternewman closed 4 years ago

peternewman commented 4 years ago

This is sufficient to dog food: https://github.com/codespell-project/codespell/blob/ac9e18a3ecd7021d2dbbea40f4c0a30a5d5d449d/.github/workflows/codespell.yml#L13

peternewman commented 4 years ago

@per1234 would you mind taking a look at my first new bats test (now it's working)?

larsoner commented 4 years ago

CIs are failing

peternewman commented 4 years ago

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...

peternewman commented 4 years ago

I think this should be merge-able now @larsoner .

peternewman commented 4 years ago

Having debugged it (grumble @per1234 :stuck_out_tongue_winking_eye: ) it's because our behaviour for hidden files in directories has changed.