-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Make LSP client proxy support dynamicRegistration for client capabilities #9797
Comments
@dkuleshov Is this a blocker for Golang server? jdt.ls also supports dynamic capabilities however if a client does not support it the server should be able to fall-back to fixed capability reporting. If this is not the case for Golang we should report it. |
@gorkem According to what I see golang LS ignores I'm not yet sure if that's a blocker for #9781 but if it is than it will be mentioned here as well. |
Will take to the nearest sprint for analyzing this ability for our client |
@vparfonov Do we really need this supported on the GWT IDE? LS implementations are supposed to work without it. |
I think it's just nice to have, not critical. Firstly we need to upgrade lsp4j (#9864), and fix several technical impediments. Which mainly comes from our feature for supporting multi language server for on opened file. |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Actual implementation does not support dynamic client capabilities registration. In order to make the workflow more flexible we should support it. Seems like depending on the implementation some servers can provide capabilities dynamically (e.g. Golang server).
See
WorkspaceClientCapabilities
andTextDocumentClientCapabilities
in LSP Specification.The text was updated successfully, but these errors were encountered: