Skip to content

docs: Mention support for alternate LSP for Kotlin - #45650

Merged
yeskunall merged 2 commits into
zed-industries:mainfrom
ilsubyeega:patch-1
Feb 9, 2026
Merged

docs: Mention support for alternate LSP for Kotlin#45650
yeskunall merged 2 commits into
zed-industries:mainfrom
ilsubyeega:patch-1

Conversation

@ilsubyeega

@ilsubyeega ilsubyeega commented Dec 25, 2025

Copy link
Copy Markdown
Contributor

Would believe #40776 did change the default lsp to official implementation.

Release Notes:

  • N/A

@cla-bot

cla-bot Bot commented Dec 25, 2025

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @ilsubyeega on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@ilsubyeega

Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Dec 25, 2025
@cla-bot

cla-bot Bot commented Dec 25, 2025

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@yeskunall yeskunall self-assigned this Dec 25, 2025
@yeskunall yeskunall changed the title Update trivial Kotlin documentation docs: Update trivial Kotlin documentation Dec 25, 2025

@yeskunall yeskunall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should instead be updated here directly https://github.com/zed-extensions/kotlin/blob/main/README.md as you linked in your change, and not here.

@yeskunall

yeskunall commented Feb 9, 2026

Copy link
Copy Markdown
Member

@ilsubyeega also not sure what you’ve said is true. https://github.com/zed-extensions/kotlin/blob/main/src/kotlin.rs#L27-L54 tells me that the extension still picks up the first match, which if you have kotlin-language-server, it will pick that up. #40776 means that if you have both LSPs installed, then kotlin-lsp will take preference.

To use Kotlin LSP instead of the Kotlin Language Server, you must explicity enable it in your settings.json:

This line in the README needs to change as with #40776, that’s the default. Hope what I’ve said makes sense.

@ilsubyeega

Copy link
Copy Markdown
Contributor Author

This should instead be updated here directly

I mean, Most user would prefer to see the document from searchable with google, search result zed kotlin #1 shows the zed.dev/docs site.

@ilsubyeega also not sure what you’ve said is true. [...]

I wrote this down for verification because, at that time when I was working on it, zed tried to run kotlin-language-server rather than kotlin-lsp. Since it's NixOS, the environment is quite complicated, so you can skip this part, I don't want to revisit that too. The point is to make users aware of the existence of kotlin-lsp.

@yeskunall

Copy link
Copy Markdown
Member

I mean, Most user would prefer to see the document from searchable with google, search result zed kotlin #1 shows the zed.dev/docs site.

On second thought, yeah you’re right. I’ve updated the copy to match our docs for other languages.

Thanks for the suggestion!

@yeskunall yeskunall changed the title docs: Update trivial Kotlin documentation docs: Mention support for alternate LSP for Kotlin Feb 9, 2026
@yeskunall
yeskunall enabled auto-merge (squash) February 9, 2026 21:49
@yeskunall
yeskunall merged commit 81f9354 into zed-industries:main Feb 9, 2026
29 checks passed
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
)

Would believe zed-industries#40776 did
change the default lsp to official implementation.


Release Notes:

- N/A

---------

Co-authored-by: Kunall Banerjee <hey@kimchiii.space>
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
)

Would believe zed-industries#40776 did
change the default lsp to official implementation.


Release Notes:

- N/A

---------

Co-authored-by: Kunall Banerjee <hey@kimchiii.space>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants