search

dundalek / lazy-lsp.nvim

Neovim plugin to auto install LSP servers
MIT License
203 stars 17 forks source link

feat: Allow preferring local lsp #30

Closed marcusramberg closed 9 months ago

marcusramberg commented 9 months ago

Note: I also cleaned up some markdown in README.md, I'd be happy to revert if you prefer the old version.

Fixes #22

dundalek commented 9 months ago

Thanks, looks good. The vim.fn.executable is neat.

I will need to figure out how to write a test for it to add to existing tests since it is not a pure computation but relies on the state of the underlying system.

As a note to myself it feels process_config takes too many parameters now. It will probably need to get refactored to take a table of options. But I can do that after merge later (relying on test coverage to make sure refactoring won't break anything).