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

Using spellwarn:disable-next-line creates new diagnostic #1

Closed
NStefan002 opened this issue Jun 7, 2024 · 3 comments
Closed

Using spellwarn:disable-next-line creates new diagnostic #1

NStefan002 opened this issue Jun 7, 2024 · 3 comments

Comments

@NStefan002
Copy link

Nice plugin! Just one thing, using spellwarn:disable-next-line creates a diagnostic for that line. So basically I disabled one diagnostic just for another one to pop up. I only use the basic functionality of spell (just z=), so I don't know if there is a way to add spellwarn to the dictionary.

On the other hand spellwarn:disable for disabling spellwarn on entire file works just as respected.

image

@ravibrock
Copy link
Owner

ravibrock commented Jun 7, 2024

Hey! I'm glad you like the plugin. That's intended behavior (at least at the moment), since I elected to not change any of Neovim's spelling settings. If you put your cursor onto spellwarn and hit zg it will mark spellwarn as being spelled correctly.

Later today I can look into updating spellwarn to disable diagnostics on lines containing disable-next-line as well.

@NStefan002
Copy link
Author

That's intended behavior (at least at the moment), since I elected to not change any of Neovim's spelling settings.

That's reasonable.

On the other hand spellwarn:disable for disabling spellwarn on entire file works just as respected.

I totally butchered this sentence, ignore it... I meant to say there is an inconsistency between disable-next-line and the other two options, because disable-next-line produces a warning.

Later today I can look into updating spellwarn to disable diagnostics on lines containing disable-next-line as well.

That's exactly what I was going for.

zg it will mark spellwarn as being spelled correctly

Thanks for the tip, btw

@ravibrock
Copy link
Owner

Added commit ababc71 to not show diagnostics for the word spellwarn.

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