Skip to content

Conversation

@fjl
Copy link
Contributor

@fjl fjl commented Feb 20, 2019

This adds the disableFuncSnippet initialization option which is recognized by the bingo
language server. See https://github.com/saibing/bingo/blob/master/langserver/initialization.go for supported initialization options.

This adds the disableFuncSnippet initialization option which is recognized by the bingo
language server.
@fjl
Copy link
Contributor Author

fjl commented Feb 20, 2019

BTW, I don't really understand why there are two ways to disable snippet completion for
Go. There is lsp-enable-snippet to announce snippet capability and lsp-client-go-func-snippet-enabled to configure the language server via initialization options.

@yyoncho
Copy link
Member

yyoncho commented Feb 20, 2019

lsp-client-go-func-snippet-enabled

This is go language server specific setting while lsp-enable-snippet is the standard setting in lsp protocol - refer to the spec https://microsoft.github.io/language-server-protocol/specification

@yyoncho
Copy link
Member

yyoncho commented Feb 20, 2019

Thank you for the PR and congrats for your first lsp-mode PR! 🎉

@yyoncho yyoncho merged commit eb8637c into emacs-lsp:master Feb 20, 2019
@fjl
Copy link
Contributor Author

fjl commented Feb 20, 2019

I'll try to also fix the bingo server so it doesn't send snippets if they're not announced in client capabilities.

waymondo pushed a commit to waymondo/lsp-mode that referenced this pull request Feb 26, 2019
This adds the disableFuncSnippet initialization option which is recognized by the bingo
language server.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants