Skip to content
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

ALE does not specify capabilities on initialize request #2515

Closed
luizribeiro opened this issue May 20, 2019 · 3 comments
Closed

ALE does not specify capabilities on initialize request #2515

luizribeiro opened this issue May 20, 2019 · 3 comments

Comments

@luizribeiro
Copy link

Information

VIM version

NVIM v0.3.5
Build type: Release

Operating System: macOS 10.14.5

What went wrong

I've been working on integrating ALE with omnisharp-roslyn's LSP server.

However, I ran into a problem where omnisharp-roslyn isn't working because ALE does not specify which capabilities it supports (see OmniSharp/omnisharp-roslyn#1499).

In looking into the code, it seems like we're just not sending anything on capabilities:

https://github.com/w0rp/ale/blob/3b7c86e401eb488acd8713f7c9ee81344095080d/autoload/ale/lsp/message.vim#L32-L40

I've tried changing it to something as simple as:

    return [0, 'initialize', {
    \   'processId': getpid(),
    \   'rootPath': a:root_path,
    \   'capabilities': {
    \       'workspace': {
    \       },
    \       'textDocument': {
    \           'completion': {
    \               'completionItem': {'snippetSupport': v:false},
    \           },
    \       },
    \   },
    \   'initializationOptions': a:initialization_options,
    \   'rootUri': ale#path#ToURI(a:root_path),
    \}]

And it helped, but I'm not sure what capabilities we should specify here.

Do we have a comprehensive list of the capabilities that are supported by ALE?

@w0rp
Copy link
Member

w0rp commented May 20, 2019

I'll update it to do that.

@w0rp
Copy link
Member

w0rp commented May 20, 2019

Now ALE will send some capabilities to the servers. If the Dictionary needs tweaking, let me know.

@luizribeiro
Copy link
Author

Wow, thanks @w0rp! That was quick.

I'll give it a try tonight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants