Skip to content

Conversation

@vitallium
Copy link
Contributor

The Ruby extension uses the solargraph
language server by default for Ruby files.
However, when a user opens any ERB file,
the extension automatically starts the Ruby LSP.
This affects developers because
they do not expect the Ruby LSP to be running.

Closes zed-extensions/ruby#172

Release Notes:

  • N/A

The Ruby extension uses the `solargraph`
language server by default for Ruby files.
However, when a user opens any ERB file,
the extension automatically starts the Ruby LSP.
This affects developers because
they do not expect the Ruby LSP to be running.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Nov 2, 2025
@osiewicz osiewicz merged commit 222767e into zed-industries:main Nov 3, 2025
25 checks passed
@marcoroth
Copy link
Contributor

marcoroth commented Nov 3, 2025

@vitallium I wonder if this is going to be breaking features like this: Shopify/ruby-lsp-rails#659

/cc @janko

@vitallium
Copy link
Contributor Author

@vitallium I wonder if this is going to be breaking features like this:

https://bsky.app/profile/janko.io/post/3m4bqpfxsvs2v

/cc @janko

Yes, it will break such things, so we need to document how to enable Ruby LSP in erb file, I think. I will open a pull request for that. Thanks for the heads up, Marco!

@vitallium vitallium deleted the vs/disable-ruby-lsp-for-erb-files-by-default branch November 3, 2025 20:08
@janko
Copy link
Contributor

janko commented Nov 4, 2025

Since I already have to explicitly enable ruby-lsp for ruby files, it makes sense to me that I have to do the same for ERB files 👍🏻

P1n3appl3 pushed a commit to bnjjj/zed that referenced this pull request Nov 4, 2025
The Ruby extension uses the `solargraph`
language server by default for Ruby files.
However, when a user opens any ERB file,
the extension automatically starts the Ruby LSP.
This affects developers because
they do not expect the Ruby LSP to be running.

Closes zed-extensions/ruby#172

Release Notes:

- N/A
osiewicz pushed a commit that referenced this pull request Nov 5, 2025
Hi, this is a follow-up change for
#41754 I think it important to
keep existing things working. So add notes to the Ruby extension doc
about enabling Ruby LSP for ERB files as well. Thanks!

Release Notes:

- N/A
maxbrunsfeld pushed a commit that referenced this pull request Nov 6, 2025
Hi, this is a follow-up change for
#41754 I think it important to
keep existing things working. So add notes to the Ruby extension doc
about enabling Ruby LSP for ERB files as well. Thanks!

Release Notes:

- N/A
maxbrunsfeld pushed a commit that referenced this pull request Nov 6, 2025
Hi, this is a follow-up change for
#41754 I think it important to
keep existing things working. So add notes to the Ruby extension doc
about enabling Ruby LSP for ERB files as well. Thanks!

Release Notes:

- N/A
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 community champion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ruby-LSP and Rubocop cannot be disabled

4 participants