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

Export an API to indicate when lsp4mp has started #58

Closed

Conversation

datho7561
Copy link
Contributor

Instead of returning from activate when lsp4mp has started, return before it has started. This allows extensions that depend on vscode-microprofile to begin activating before lsp4mp has fully started. Return an API that provides a promise which resolves once lsp4mp has started, so that extensions that need to interact with lsp4mp can wait until it is ready.

Signed-off-by: David Thompson [email protected]

Instead of returning from activate when lsp4mp has started,
return before it has started.
This allows extensions that depend on vscode-microprofile to begin
activating before lsp4mp has fully started.
Return an API that provides a promise which resolves once lsp4mp has
started, so that extensions that need to interact with lsp4mp can wait
until it is ready.

Signed-off-by: David Thompson <[email protected]>
datho7561 added a commit to datho7561/vscode-quarkus that referenced this pull request Jun 28, 2021
See redhat-developer/vscode-microprofile#58

Consume the API provided by vscode-microprofile in order to
wait for lsp4mp to start when running commands that interact with
lsp4mp.

Closes redhat-developer#323

Signed-off-by: David Thompson <[email protected]>
@datho7561 datho7561 force-pushed the 323-exit-activate-earlier branch from 38d88e5 to 2bc47b8 Compare June 29, 2021 14:09
@angelozerr
Copy link
Contributor

@datho7561 what is the state of this PR?

@datho7561
Copy link
Contributor Author

This strategy doesn't actually address the issue I was trying to solve, which is needing to activate vscode-java in order to use the project generator in vscode-quarkus. This PR means that you can start vscode-microprofile and vscode-quarkus in parallel though, so it cuts down on startup time. I think Fred mentioned that he preferred the other approach that I tried, so I'll close this PR.

@datho7561 datho7561 closed this Aug 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants