antonk52 / cssmodules-language-server

autocompletion and go-to-defintion for cssmodules
MIT License
54 stars 8 forks source link

Change nomenclature after Neovim 8 #12

Closed kecerud closed 1 year ago

kecerud commented 1 year ago

There have been changes in neovim 8 see: https://github.com/neovim/nvim-lspconfig/issues/1891#issuecomment-1157964108

Updated part of README to match the current property used in neovim 8.

kecerud commented 1 year ago

Actually goto_definition seems deprecated as well? Or I just don't see it in the server_capabilities options (:lua =vim.lsp.get_active_clients()[1].server_capabilities) 🤔

antonk52 commented 1 year ago

Thanks for the PR, it looks like they've changed the name from goto_definition to definitionProvider. I've just tested it and it works it used for resolved_capabilities. goto_definition.

kecerud commented 1 year ago

No problem, I wasn't sure about that so changed that one as well :) Should be good to go now

antonk52 commented 1 year ago

Thank you for keeping documentation updated! Merged 🙂