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 option to only spellcheck comments #36

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

Conversation

alok
Copy link

@alok alok commented Oct 18, 2017

The user may not want to spellcheck strings (I don't, anyway). If they set
g:python_spellcheck_strings = 0, then they can accomplish that. This also
closes #12.

@alok
Copy link
Author

alok commented Oct 18, 2017

I thought some use of exe and variables would clean up the code, but the amount of regex escaping made me decide against that.

@nfnty
Copy link
Member

nfnty commented Oct 18, 2017

This will create an unmaintainable mess. There has to be a better way of implementing this. Is it possible to make @Spell conditional with syntax groups? Otherwise execute is the better option; regex doesn't need to be escaped when using single quotes.

@alok
Copy link
Author

alok commented Oct 18, 2017

Hmm, I'd prefer to entirely remove spell checking strings since they aren't really meant to correspond strictly to actual human language (e.g keys in a dict like a, b, c).

Otherwise, I could implement something with execute. I had another branch with some work on that.

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

Successfully merging this pull request may close these issues.

Make @Spell configurable for strings
2 participants