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

Response to the server request workspace/configuration #1081

Merged

Conversation

jdneo
Copy link
Collaborator

@jdneo jdneo commented Sep 29, 2019

More details: eclipse-jdtls/eclipse.jdt.ls#1194

Signed-off-by: Sheng Chen [email protected]

@snjeza
Copy link
Contributor

snjeza commented Sep 29, 2020

@jdneo could you, please, rebase the PR?

src/extension.ts Outdated
languageClient.onRequest(ConfigurationRequest.type, (params: ConfigurationParams) => {
const result: any[] = [];
const activeEditor: TextEditor | undefined = window.activeTextEditor;
for (const item of params.items) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should send java.format.insertSpaces and java.format.tabSize to align with https:/eclipse/eclipse.jdt.ls/pull/1551/files#diff-3feb2f2158f1898f0a00165abf820e3dR69-R73

@jdneo
Copy link
Collaborator Author

jdneo commented Sep 29, 2020

Sure, will rebase it later today.

@testforstephen testforstephen added this to the End February milestone Feb 24, 2021
@testforstephen testforstephen merged commit d91377f into redhat-developer:master Feb 24, 2021
@jdneo jdneo deleted the cs/workspace/configuration branch February 24, 2021 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants