sublimelsp / LSP

Client implementation of the Language Server Protocol for Sublime Text
https://lsp.sublimetext.io/
MIT License
1.64k stars 182 forks source link

[regression] LSP fails to initialize #655

Closed ayoub-benali closed 5 years ago

ayoub-benali commented 5 years ago

With this commit LSP fails to initialize, we can see it in the console (enable debug):

LSP: global configs ['ocaml=False', 'vscode-css=False', 'haskell-ide-engine=False', 'dart=False', 'polymer-ide=False', 'javascript-typescript-langserver=False', 'metals=False', 'intelephense-ls=False', 'ruby=False', 'jdtls=False', 'golsp=False', 'gopls=False', 'bashls=False', 'typescript-language-server=False', 'lsp-tsserver=False', 'pyls=False', 'clangd=False', 'phpls=False', 'rls=False', 'reason=False', 'eslint=False', 'flow=False', 'ra-lsp=False', 'cquery=False']
LSP: window 5 has override for metals {'enabled': True}
LSP: window 5 starting 2 initial views
LSP: window 5 requests metals for ****
LSP: starting in ****
LSP: starting ['metals-sublime']
LSP:  --> initialize
LSP: window 5 added session metals
reloading settings Packages/User/PackageSync.sublime-settings
LSP: Message could not be parsed.

I guess the initialize json got broken since the commit

  1. check out the mentioned commit hash
  2. Open sublime
  3. notice no LSP functionality will work
arsham commented 5 years ago

I'm also getting this message:

LSP: json: cannot unmarshal array into Go struct field .valueSet of type protocol.SymbolKind
ayoub-benali commented 5 years ago

@arsham issue is fixed on master. Just do a git pull

tomv564 commented 5 years ago

Whoops, thanks guys!