redhat-developer / quarkus-ls

Language server for Quarkus tooling
Eclipse Public License 2.0
43 stars 15 forks source link

Don't implement by default custom language client API #870

Closed angelozerr closed 1 year ago

angelozerr commented 1 year ago

When a LSP client consumes the last version of Qute LS, it doesn't know which new API methods that it needs to implement because all API are implemented by default which return a CompletableFuture.completableFuture(null).

We need to remove default implementation to force all LSP client (like IJ Quarkus, JBoss Tools Quarkus) to implement new methods.

//cc @sbouchet

angelozerr commented 1 year ago

Thanks so much @datho7561 for your review, I need to do the same thing for MicroProfile LS.