kickstart.nvim icon indicating copy to clipboard operation
kickstart.nvim copied to clipboard

fix(lsp): updated config for mason to v2

Open Ktsierra opened this issue 6 months ago • 4 comments

This pull request updates the LSP configuration to align with the changes in mason-lspconfig.nvim v2.0.0.

The handlers feature has been removed from mason-lspconfig.nvim. The PR adapts to that change by iterating through the servers table and calling require('lspconfig')[server_name].setup(server_config) for each server. This approach maintains the use of the nvim-lspconfig API for server configuration.

This change simplifies the LSP configuration and ensures compatibility with the latest version of mason-lspconfig.nvim.

Ktsierra avatar Jul 23 '25 18:07 Ktsierra

This seems to do a similar thing as 1590 which already has some traction

evanlesmez avatar Aug 01 '25 05:08 evanlesmez

This seems to do a similar thing as 1590 which already has some traction

There are a couple of similars pr, but 1590 and 1475 both swap to the new LSP built in API besides fixing mason, which is only availabe in version >0.11. I think is the same reason we still have blink and lazy instead of using vim.lsp and vim.pack.

Ktsierra avatar Aug 01 '25 11:08 Ktsierra

This seems to do a similar thing as 1590 which already has some traction

Traction does not mean quality.

There are a couple of similars pr, but 1590 and 1475 both swap to the new LSP built in API besides fixing mason, which is only availabe in version >0.11. I think is the same reason we still have blink and lazy instead of using vim.lsp and vim.pack.

I don't like them. Both state a different goal than they are achieving. Both are pushing their authors preferences.

On the other hand this one does exactly what it says and does it correctly.

logrusx avatar Aug 03 '25 17:08 logrusx

I just found out this repo is at best inactive: #1627

logrusx avatar Oct 15 '25 13:10 logrusx