Nvim-lspconfig: rename to nvim-lspconfig

[justinmk]

Probably should have anticipated this, but people are confused about nvim-lsp (this repo, not Nvim-builtin) vs vim.lsp (Nvim builtin). Although the nvim-lsp README.md makes it clear, it still confuses people in discussions (example), and people who are just getting started.

So we should

• rename this repo to nvim-lspconf nvim-lsp.configs
• it also makes sense to rename the Lua module: it is confusing that require'nvim_lsp' is distinct from vim.lsp, since their names are semantically equivalent
  
  
  
  • see https://github.com/neovim/nvim-lsp/issues/97#issuecomment-583717327
  
  

I will do so in the next ~week if there are no objections.

cc @norcalli

[norcalli]

No objections. I'm curious if github will handle redirection for cloning, though. It's probably worth a test run on a dummy repo.

[h-michael]

@norcalli
It automsticaly redirects.
https://help.github.com/en/github/administering-a-repository/renaming-a-repository

[teto]

nvim-lspconf or nvim-lsp-configs since there are several.
The sooner the better.

[justinmk]

Changed my mind again (x2): I am thinking:

• repo name: neovim/nvim-lspconfig
• Lua module: lspconfig
  
  
  
  • and that's where the configs singleton lives instead of wherever it lives right now.
  
  
• The various configs live in lua/lsp/configs/*.lua

I didn't look closely, but it seems like there is no reason for nvim_lsp.lua to be a separate module. It should just be lua/lsp/configs.lua module.

[teto]

agreed, the split looks a bit artificial and forces to submit changes across the 2 repo. Having really just the configs here would be best.

[teto]

@h-michael do you feel like tackling https://github.com/neovim/nvim-lsp/issues/97#issuecomment-583717327 ?

[h-michael]

@teto I'll try this this weekend.

[justinmk]

Repo is now renamed from neovim/nvim-lsp to neovim/nvim-lspconfig .

Haven't done any code changes yet. Planning to rename the Lua module to lspconfig. This clearly distinguishes it from lsp while staying simple. see https://github.com/neovim/nvim-lsp/issues/97#issuecomment-583717327