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

double formatters for java #207

Closed
maxandersen opened this issue Feb 9, 2020 · 1 comment
Closed

double formatters for java #207

maxandersen opened this issue Feb 9, 2020 · 1 comment
Labels
duplicate This issue or pull request already exists

Comments

@maxandersen
Copy link
Collaborator

latest version of quarkus tools asks me to choose wether want to use quarkus or vscode-java for java formatting.

That shouldn't happen should it ?

I tried make a screenshot but it disappeared when trying to grab it.

@angelozerr angelozerr added the duplicate This issue or pull request already exists label Feb 10, 2020
@angelozerr
Copy link
Contributor

Indeed there is a problem with java, the problem comes fromMicriProfile LS which maneges java and application.properties both. The server returns the format capability because application.properties can be formatted, but returning this format capability provides trouble with java althouth MicroProfile LS can just formatting application.properties files.

There are 2 solutions to fix this java format problem:

  • Separate java and application.properties support in 2 language server.
  • Improve JDT LS to override features like hover, codelens, diagnostics, etc

We need to discuss about that with @fbricon

I close this issue since this issue is similar to #166

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants