Skip to content

Rename volar to vue_ls and remove vuels - #3843

Merged
justinmk merged 2 commits into
neovim:masterfrom
jak-ch-ll:rename-volar
May 21, 2025
Merged

Rename volar to vue_ls and remove vuels#3843
justinmk merged 2 commits into
neovim:masterfrom
jak-ch-ll:rename-volar

Conversation

@jak-ch-ll

@jak-ch-ll jak-ch-ll commented May 11, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #3826

A few questions:

  1. Is there a good way with the new configs to alias an older name, so it stays available?
    • I found the aliases table, but this seems to only affect the old configs.
    • For the new configs I kept the lsp/volar.lua file, added a vim.deprecate, and returned vim.lsp.config.vue_ls
  2. What is a good version to remove it later? I used 3.0.0 as placeholder

Additionally (even though not super necessary), I couldn't check the generated docs because nvim -R -Es +'set rtp+=$PWD' +'luafile scripts/docgen.lua' fails without message for me

  • Running luafile scripts/docgen.lua' inside of Neovim (even after adding the path to rtp) fails with the error scripts/docgen.lua:136: module 'lsp.ada_ls' not found
  • Do I need to setup anything locally?

@jak-ch-ll jak-ch-ll mentioned this pull request May 11, 2025
Closed
@jak-ch-ll
jak-ch-ll force-pushed the rename-volar branch 2 times, most recently from ad90d68 to ad73ffa Compare May 11, 2025 13:53
jak-ch-ll added 2 commits May 11, 2025 15:54
@jak-ch-ll
feat(volar)!: rename to 'vue_ls'
221bc7b 
The Vue language tooling does not use 'Volar' prominently in its
documentation. The official name for the language server is
'@vue/language-server'.
@jak-ch-ll
chore(vuels): remove config
5b7f09a 
This project is no longer active. Use [vue_ls](lsp/vue_ls.lua) instead.
@jak-ch-ll

jak-ch-ll commented May 11, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Removed my changes to the old configs, as per lint rules

@justinmk

justinmk commented May 21, 2025

Copy link
Copy Markdown
Member
  • For the new configs I kept the lsp/volar.lua file, added a vim.deprecate, and returned vim.lsp.config.vue_ls

exactly 👍

@justinmk

justinmk commented May 21, 2025

Copy link
Copy Markdown
Member
  • Running luafile scripts/docgen.lua' inside of Neovim (even after adding the path to rtp) fails with the error scripts/docgen.lua:136: module 'lsp.ada_ls' not found
  • Do I need to setup anything locally?

that's strange. shouldn't happen

@justinmk
justinmk merged commit 2864899 into neovim:master May 21, 2025
@mehalter mehalter mentioned this pull request May 21, 2025
Merged
3 tasks done
@gx089 gx089 mentioned this pull request May 22, 2025
Closed
@dragunovartem99 dragunovartem99 mentioned this pull request Jun 7, 2025
Closed
@wvffle wvffle mentioned this pull request Aug 4, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Rename volar and sunset vuels

2 participants

@jak-ch-ll @justinmk