-
Notifications
You must be signed in to change notification settings - Fork 397
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mention change to default_capabilities() in breaking changes issue #231 #1265
Comments
Agreed. see hrsh7th/cmp-nvim-lsp#38 as well |
wookayin
added a commit
to wookayin/nvim-cmp
that referenced
this issue
Nov 16, 2023
Using `require('cmp_nvim_lsp').default_capabilities()` alone will be not enough. We have to merge with the default LSP capabilities producced by `vim.lsp.protocol.make_client_capabilities()`. See also hrsh7th/cmp-nvim-lsp#64 Closes hrsh7th#1265
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I can not comment on #231 but I think the change to
default_capabilities()
should be mentioned there.The text was updated successfully, but these errors were encountered: