Skip to content

Conversation

janko
Copy link
Contributor

@janko janko commented Oct 16, 2025

The built-in Tailwind language already maps HTML+ERB to erb, and it seems that Ruby files work as well just from enabling the language server, so we can remove the unnecessary mapping.

The built-in Tailwind language already maps `HTML+ERB` to `erb`, and it
seems that `Ruby` files work as well just from enabling the language server,
so we can remove the unnecessary mapping.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Oct 16, 2025
@janko
Copy link
Contributor Author

janko commented Oct 16, 2025

Maybe @vitallium or @joeldrapper could confirm whether it still works for them without the mappings?

@zed-industries-bot
Copy link

Warnings
⚠️

This PR is missing release notes.

Please add a "Release Notes" section that describes the change:

Release Notes:

- Added/Fixed/Improved ...

If your change is not user-facing, you can use "N/A" for the entry:

Release Notes:

- N/A

Generated by 🚫 dangerJS against 7ee925b

@nia-e nia-e merged commit 620df0c into zed-industries:main Oct 16, 2025
23 checks passed
xipeng-jin pushed a commit to xipeng-jin/zed that referenced this pull request Oct 17, 2025
…d-industries#40299)

The built-in Tailwind language already maps `HTML+ERB` to `erb`, and it
seems that `Ruby` files work as well just from enabling the language
server, so we can remove the unnecessary mapping.

Release Notes:
- N/A
P1n3appl3 pushed a commit to P1n3appl3/zed that referenced this pull request Oct 18, 2025
…d-industries#40299)

The built-in Tailwind language already maps `HTML+ERB` to `erb`, and it
seems that `Ruby` files work as well just from enabling the language
server, so we can remove the unnecessary mapping.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants