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
fix: support volar 1.0 #3787
fix: support volar 1.0 #3787
Conversation
Can you add changelog entry? That will break the config of those using the old server right? |
28bbd52
to
a17cf8e
Compare
Can you resolve the conflicts? Then it is good to go |
a17cf8e
to
5a81f39
Compare
Thank you! |
:initialization-options (lambda () (ht-merge (lsp-configuration-section "typescript") | ||
(lsp-configuration-section "documentFeatures"))) | ||
(ht ("serverMode" 0) | ||
("diagnosticMode" 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here are a wrong field: diagnosticMode
-> diagnosticModel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your correction, I will fix it.
Volar has update the options for the language server. See vuejs/language-tools#1916 .
And fix #3785