-
Notifications
You must be signed in to change notification settings - Fork 18
feature: emmylua-analyzer support #86
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
Comments
Interesting for sure! |
Added very basic support. Pretty sure it won't work as is though :) Didn't test it either. Would be great to have it in lspconfig so it's easier to test. |
FWIW, I'm testing with a vim.lsp.config("*", {
root_markers = { ".git" },
})
local should_enable = require("helpers.lsp").should_enable
vim.iter(vim.api.nvim_get_runtime_file("lsp/*.lua", true))
:map(function(config_path)
return vim.fs.basename(config_path):match("^(.*)%.lua$")
end)
:each(function(server_name)
vim.lsp.enable(server_name, should_enable(server_name))
end) |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Not stale |
note: you'll have to download https://github.com/LuaCATS/luv and provide the corresponding path to emmyrc's |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Not stale |
Looks like this LSP has been added to nvim-lspconfig here neovim/nvim-lspconfig#3745 |
Did you check the docs?
Is your feature request related to a problem? Please describe.
One of the developers for lua-language-server has published a rust re-implementation, called emmylua-analyzer.
Describe the solution you'd like
It uses a different format for its configuration, but afaict, it supports the same
workspace.library
aslua-language-server
. Currently, it only makes theworkspace/configuration
request when it detects VSCode, and it doesn't support setting theworkspace.library
from theworkspace/configuration
call. I've opened an issue about these in the emmylua-analyzer repo: EmmyLuaLs/emmylua-analyzer-rust#6Given support is added, would you be interested in supporting it?
Describe alternatives you've considered
Creating a separate plugin specifically for emmylua-analyzer but I feel this would cause unnecessary fragmentation
Additional context
No response
The text was updated successfully, but these errors were encountered: