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

Add spell check #47

Open
AlexAxthelm opened this issue Feb 8, 2024 · 2 comments
Open

Add spell check #47

AlexAxthelm opened this issue Feb 8, 2024 · 2 comments

Comments

@AlexAxthelm
Copy link
Contributor

I go back and forth on if this is a good idea or not.

I find CSpell useful while coding, since it can work on both normal prose (.md, .txt, etc) and on code files (.R, etc) while being pretty good about dealing with coding conventions, like snake case.

https://cspell.org/docs/how-it-works/ lays out:

The concept is simple, split camelCase and snake_case words before checking them against a list of known words.

which is great, but leads to my single biggest frustration when dealing with CSpell: maintaining the custom word list. Overall, it's not that difficult, since CSpell traverses up the directory tree until to find custom dictionaries, which means keeping a wordlist in the repo root, and updating it when new words are added to the repo.

I don't have any benchmark as to how any of our repos fare against the standard dictionaries

@jdhoffa
Copy link
Member

jdhoffa commented Feb 8, 2024

Ooof I would be very hesitant to enforce this at all. I think people should use if they want, but I had CSpell enforced in my local neovim config for a bit and hated it.

I spent 99% of my time adding things to the exception list and I eventually just removed it

Recognizing that that may have been an issue with HOW i configured it

@jdhoffa
Copy link
Member

jdhoffa commented Feb 8, 2024

Ah, but maybe as you say only enforcing in .md could be good?

That would even catch typos in documented R package functions

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

No branches or pull requests

2 participants