fix(lvim/lsp/manager): make client_is_configured more reliable #2851
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
A server_name can be the same as its filetype, which will match the given pattern if you set any filetype autocmd for it. This might prevent a server from attaching.
Consider the html lsp. Its server_name is simply
html
. This might prevent a user from setting up plugins likenvim-colorizer
,nvim-ts-autotag
or using the emmet lsp in tandem.Solution:
Use the lua API so we can match the specific command string, which contains only the manager's setup call, instead of trying to match/parse the result of
:autocmd
.How Has This Been Tested?
:LvimInfo
.