-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
support vscode.CustomExecution api #7185
Comments
I don't think we support |
Version https://github.com/redhat-developer/vscode-java/releases/download/v0.54.2/redhat.java-0.54.2.vsix |
@marcdumais-work I test |
Is implementation of CustomExecution planned? |
Contributions to issues marked as |
Someone from my team will take a look at this after the upcoming Che sprint, so sometime late August. Of course by no means should that stop someone else from working on it before then, if they want to! |
Hey @ericwill Some info: I'm running theia 1.6.0, with However, I see this error when the extension starts:
Are you guys working on this? Anything I could do as a workaround to make it work? |
@gsmachado no, it's not in the sprint plan. We're aware that this API is not implemented. It's a question of assigning resources. |
Correct. @gsmachado if you propose a PR to theia, someone from our team can review it. |
Alright! I would be interested to do it, but I'm not even sure what's the correct behavior of this I would appreciate any leads on this -- maybe I get inspired and get something done. 😄 |
Hi @gsmachado, A few pointers - those around here that know more than me can correct or complete: vscode API doc: https://code.visualstudio.com/api/references/vscode-api#CustomExecution implementation: On the Theia side: And probably to enhance this to look more like vscode above: examples of the API being used by P.S. We generally do not reinvent the wheel. It's permissible to copy code from the |
@marcdumais-work awesome answer. You know, being in the open-source space for some time, that's what I believe we need more: people willing to at least explain, with a reasonable level of detail, what needs to be done, where people can start with, etc. This kind of thing motivates people. 👍 😄 |
@gsmachado I hope this is good to get you started. Please ask here if you have questions along the way. |
Well, while this API is not implemented, is there a work-around? A way for plug-in authors to implement a Task that's simply a callback, for example, even if I have to keep two code versions (one for vscode and one for Theia) Otherwise, my plug-in will not work at all :( |
@mauricioszabo I do not believe there is a workaround at the moment, if the feature is important to your use-case you can always attempt to implement the API yourself and submit a pull-request. The following #7185 (comment) describes the necessary changes in detail. |
Hi @gsmachado, Thank you! |
I will look into this. |
Signed-off-by: Akos Kitta <kittaakos@typefox.io>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <tanhaiyang@163.com>
I have an implementation,#9189 |
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
issue: eclipse-theia#7185, eclipse-theia#8767 Signed-off-by: Lewin Tan <e_tanhaiyang@163.com>
Description
To see that Java language server is initializing need to click on the task button:
But this button doesn't work:
Reproduction Steps
OS and Theia version:
Linux
@theia/example-browser 0.15.0
Diagnostics:
That's how it looks in vscode:
The text was updated successfully, but these errors were encountered: