-
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
[lsp][plug-in] Ease the integration of a new LSP into Eclipse Che with plug-ins #10206
Comments
Please no! Let's keep make it so that we can take any lsp, package it in a container and then run it as a che plugin. Special Che API would be counter to that goal. |
Maybe I don't get it: there is already api to contribute syntax highlighting in Theia. Why would we add something incompatible? |
@tsmaeder I don't know if you read too quickly but it would be only if there is something specific to che that we could require a Che API (like grabbing the url of a container, getting some container metadata, or anything else specific due to Che and Theia integration). Also AFAIK for now syntax highlighting is through extension not plug-in model. to sum-up, goal is more to provide Theia "plug-in" model to LSPs. |
@tsmaeder I've updated the description of the issue |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Description
Eclipse Che is becoming more and more extensible and customizable.
Workspace.next engine is allowing to plug new
tool container
into the workspaceTheia and plug-in model is allowing an easy way to plug new IDE components as well. (instead of Theia extension)
Goal of this epic is to get advantages of these two when we talk about LSP implementations.
Plug-in's model API should provide LSP plug-ability for the client side. (like the textmate configuration, the lsp options, etc)
Definition of the container providing LSP server should be a workspace.next configuration.
All of this should be then packaged as a Che Plugin.
Tasks
The text was updated successfully, but these errors were encountered: