Skip to content
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

Spelling #924

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Spelling #924

wants to merge 16 commits into from

Conversation

jsoref
Copy link

@jsoref jsoref commented Mar 22, 2020

This was developed using https://github.com/jsoref/piskel/tree/spell-check, you can see the runs here: https://github.com/jsoref/piskel/actions/

Historically, I'd run a private spell checking tool against a repository and then just contribute the fixes. And then a couple of years later, I'd come back and repeat the process. To the extent that I manage to get back to a project, that works. But the alternative is actually offering a tool that projects can use to catch errors in advance. If you're interested, I'm happy to make a PR w/ the GitHub action.

I'm still getting used to using this as an Action, especially the idea of telling it to ignore CSS colors -- which resulted in a bit of a goose chase trying to figure out where sper and nvas were coming from. In normal usage, when one tries to add a misspelling, the tool will annotate the new word and it's pretty obvious.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants