Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Spell checking of comments #9

Closed
ahaeger opened this issue Aug 1, 2016 · 6 comments
Closed

Spell checking of comments #9

ahaeger opened this issue Aug 1, 2016 · 6 comments
Assignees

Comments

@ahaeger
Copy link

ahaeger commented Aug 1, 2016

I usually write todos to myself, something like

% TODO Baltrunas' ideas should be explored further.

The spell checker then complains on the words "TODO" and "Baltrunas'" (which is a surname). I see the point in spell checking comments, but in this case it also kind of annoying. Maybe one should be able to turn spell checking for comments on and off?

@yitzchak yitzchak self-assigned this Aug 1, 2016
@yitzchak
Copy link
Contributor

yitzchak commented Aug 1, 2016

Do you have language-todo enabled?

@ahaeger
Copy link
Author

ahaeger commented Aug 2, 2016

Yes, not entirely sure what it does though. From what I can tell by disabling/enabling, it seems like it only highlights the "TODO" part.

@yitzchak
Copy link
Contributor

yitzchak commented Aug 2, 2016

You are correct, it just highlights TODO and various other keywords in comments. I added the scopes that are used by language-todo in AtomLinter/linter-spell@fa6073e so at least those will not be spell checked. I'll push out a patch release in a bit.

@yitzchak
Copy link
Contributor

yitzchak commented Aug 2, 2016

The patch to ignore language-todo scopes just got pushed out in linter-spell v0.9.2

I am still thinking about how to make spell checking in comments a configuration option.

@yitzchak
Copy link
Contributor

yitzchak commented Aug 3, 2016

Added a configuration option to control spell checking in comments. You will need to upgrade to linter-spell v0.10.0 and linter-spell-latex v0.8.0

@ahaeger
Copy link
Author

ahaeger commented Aug 3, 2016

I think that will solve this issue for most users.

Awesome work, keep it up!

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

No branches or pull requests

2 participants