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

Adapt LspDiagnostics* highlight groups to latest neovim upstream changes (fixes #147) #148

Merged
merged 1 commit into from
Nov 13, 2020

Conversation

ybc37
Copy link

@ybc37 ybc37 commented Nov 13, 2020

Hey there!

I adapted the highlight groups based on:

I set LspDiagnosticsDefault* and removed virtual text and floating highlight groups, because they link to default, AFAIU. Fixes #147.

Would be great if someone could review/test :) Thanks!

@rbong
Copy link

rbong commented Nov 13, 2020

Can you keep the old highlight groups for now and add the new ones, and add a comment indicating that? I'd like to not break older versions of nvim for now. Thank you!

@ybc37
Copy link
Author

ybc37 commented Nov 13, 2020

Thanks for the fast feedback :)

IMHO it's not worth to keep the previous highlight groups for compatibility, because neovims built-in LSP wasn't released yet in an official version. It only concerns people following head.

But if you still prefer to keep them I'll change it, of course! (I'm just afraid I can't do it before Sunday.)

@rbong
Copy link

rbong commented Nov 13, 2020

Ah okay, I didn't know that. That's fine then.

@rbong rbong merged commit 2636a03 into gruvbox-community:master Nov 13, 2020
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.

neovim diagnostic highlight group name change
2 participants