Spell checker checking only committed files?

[bclaremar]

No description provided.

[richelbilderbeek]

I think this is possible. I will take a look :-)

[richelbilderbeek]

Created an Issue at igsekor/pyspelling-any#8

[richelbilderbeek]

I'll do so, no need to keep Björn assigned here :-)

[richelbilderbeek]

Posted in Matrix:

Both me and @jayant-yadav think the spellcheck takes too long. A thing I do nowadays is to work on the develop branch, which only has short CI test. I merge to main when done (and then get a full-blown check). The spellcheck only runs on main and for Pull Requests 👍️

[richelbilderbeek]

I feel it is good enough, but maybe @jayant-yadav wants to spend some time on this? The feature to only check changes has been suggested to the spellcheck author.

I will assign him, so he can close the Issue when he thinks it is fixed well enough too 👍

[paran1]

I saw this issue after I made PR #143

I noticed that spellchecking is spending almost all its time checking `perl.md. I have not dug into pyspelling to figure out what kind of horrible edge-case it triggers, instead I just exclude that file from spellchecking. Then I also change to use a more updated pyspelling github actuin, and some other things, check the PR.

If #143 is merged then I think this issue can be closed as we can run spellcheck on each push for all files (except perl.md) in less than 30 seconds.

[richelbilderbeek]

@paran1 this is brilliant! Thanks 👍!