Skip to content

Conversation

@igorechek06
Copy link
Contributor

@igorechek06 igorechek06 commented Apr 13, 2025

I just read nvim-jdtls configuration example and make hypotheses about jdtls.
And it turned out, the settings field in InitializationOptions is a field in DidChangeConfigurationParams.
It is so..... stupid.

@cla-bot

This comment was marked as resolved.

@igorechek06

This comment was marked as resolved.

@cla-bot cla-bot bot added the cla-signed label Apr 13, 2025
@cla-bot

This comment was marked as resolved.

@igorechek06
Copy link
Contributor Author

This is the first time when documentation and examples lie to me.

This was linked to issues Apr 14, 2025
@valentinegb valentinegb added bug Something isn't working language-server Related to the language server. labels Apr 14, 2025
@valentinegb
Copy link
Collaborator

Whoa, this is big if you're right O.O
Have you tested this?

@igorechek06
Copy link
Contributor Author

igorechek06 commented Apr 14, 2025

Yes I did. Works great. (I tested formating only 😅)

@igorechek06
Copy link
Contributor Author

I tested some options right now. Finally I can disable auto build option. But I can't disable completion.guessMethodArguments. So weird.
"format" and "signatureHelp" also works.

@igorechek06
Copy link
Contributor Author

I find this documentation. JDTLS documentation completely broken.
Screenshot_20250414-080221

@valentinegb
Copy link
Collaborator

Oops- didn't mean to remove you as an author of the commit- if you know how, feel free to amend it and force push so that you're a co-author like I intended ^^'

@notpeter
Copy link
Contributor

Oops- didn't mean to remove you as an author of the commit- if you know how, feel free to amend it and force push so that you're a co-author like I intended ^^'

@valentinegb You can also manually add Co-Authored-By: with the info from the original commit in the description when you squash and merge.

@valentinegb valentinegb merged commit 778e6e2 into zed-extensions:main Apr 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cla-signed language-server Related to the language server.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Formatter does not work LSP server settings has no effect

3 participants