Skip to content

Conversation

@luhagel
Copy link
Contributor

@luhagel luhagel commented Nov 12, 2025

Until the zed extension handles this natively, Zed users need to add an extra arguments setting within the lsp section of their config.

@luhagel luhagel changed the title Docs: Update Zed installation instructions to include setting the --stdio flag docs(installation): Update Zed installation instructions to include setting the --stdio flag Nov 12, 2025
@luhagel luhagel changed the title docs(installation): Update Zed installation instructions to include setting the --stdio flag docs(installation): update zed installation instructions to include setting the --stdio flag Nov 12, 2025
@luhagel
Copy link
Contributor Author

luhagel commented Nov 12, 2025

Talking about the Zed Extension handling it natively: zed-extensions/elixir#45

@doorgan
Copy link
Collaborator

doorgan commented Nov 12, 2025

It seems that PR was already merged, so maybe we could update the docs here to indicate how to pass the --port <port> flag if you want to use tcp mode instead?

@luhagel
Copy link
Contributor Author

luhagel commented Nov 12, 2025

Works for me - I'd maybe consider merging this for now, as there can be a couple days to a couple weeks delay between merging and publishing, had that with HEEX support for the emmet extension a few month ago.
So maybe get the info out there now (optionally amended with the tcp docs) and then counter-commit once it's actually available to the public?

Just let me know your preferred path of action and I'm happy to oblige

(Or just scrap it, it's not like I poured hours of work into this ^^)

@doorgan
Copy link
Collaborator

doorgan commented Nov 12, 2025

there can be a couple days to a couple weeks delay between merging and publishing

Ah got it, makes sense then, let's merge this

@doorgan doorgan merged commit d80afb1 into elixir-lang:main Nov 12, 2025
36 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants