AstroNvim / AstroNvim/astrocommunity
Simplify Vue LSP config
- Dominant language
- Lua
- Stars
- 1.7k
- Forks
- 313
- PR merge metrics
- No merged PRs in 30d
Description
### Is your feature related to a problem?
Simpler setup and less for us to maintain.
### Describe the new feature
At the moment we have https://github.com/AstroNvim/astrocommunity/blob/d451f1d8e90e21c0cad730add858d8f952c66f01/lua/astrocommunity/pack/vue/init.lua#L21 this manual setup copied from https://github.com/vuejs/language-tools/wiki/Neovim
It's similar to what's offered upstream in lspconfig: https://github.com/neovim/nvim-lspconfig/blob/f47cd681d7cb6048876a2e908b6d8ba1e530d152/lsp/vue_ls.lua#L25
I think it's possible to remove what we have and rely on their config.
### Additional context
Need to make sure that the [lspconfig version](https://github.com/neovim/nvim-lspconfig/releases) shipped with astronvim has the mentioned config.
Contributor guide
Research direction
Read lua/astrocommunity/pack/vue/init.lua and compare its manual setup with the linked upstream lspconfig lsp/vue_ls.lua. Check the lspconfig version shipped with AstroNvim against the release containing the Vue config. Done means the duplicated setup is removed and Vue LSP still works through the upstream configuration.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100