nvim-oxi icon indicating copy to clipboard operation
nvim-oxi copied to clipboard

Support for `vim.lsp`

Open noib3 opened this issue 1 year ago • 7 comments

noib3 avatar Jul 29 '22 21:07 noib3

Seems like these would require bindings directly to lua by fetching globals, since a lot of the LSP/Diagnostics stuff is implemented in Lua. Would you be open to someone contributing that? That's the approach I was taking with nvim-utils, but honestly I'd rather contribute those features to oxi and archive utils as it's not getting much use and oxi provides a better (and faster) API.

willothy avatar Jul 11 '23 20:07 willothy

Seems like these would require bindings directly to lua by fetching globals, since a lot of the LSP/Diagnostics stuff is implemented in Lua.

Yes, that'd be the approach. This is a very rough sketch of such a binding that I wrote for vim.diagnostic.enable.

Would you be open to someone contributing that?

Yes, although I should mention that I don't really have the time to focus on oxi right now (and probably won't for a few months). What this means practically is that you might have to wait a few days to receive feedback on the relative PRs.

noib3 avatar Jul 13 '23 00:07 noib3

No worries! I don't actively need nvim LSP features in Rust so there's no rush, just think it would be nice to have and would be happy to contribute :)

willothy avatar Jul 13 '23 01:07 willothy

I should also mention that I was planning to rewrite Neovim's vim.lsp + nvim-lspconfig + lspsaga.nvim + any-lsp-related-stuff-that-should-be-builtin-instead-of-being-relegated-to-a-3rd-party-plugin in Rust as part of a new project, hopefully before EOY.

However, this is not to say that I wouldn't merge the Rust bindings to vim.lsp into oxi.

noib3 avatar Jul 13 '23 23:07 noib3

Cool! Would be nice to have those implemented in Rust.

willothy avatar Jul 14 '23 00:07 willothy