Skip to content
This repository was archived by the owner on Jan 17, 2026. It is now read-only.

Elixir Syntax Highlighting Update - #95

Merged
Blond11516 merged 5 commits into
lexical-lsp:mainfrom
RisPNG:elixir-syntax-update
Jul 30, 2025
Merged

Elixir Syntax Highlighting Update#95
Blond11516 merged 5 commits into
lexical-lsp:mainfrom
RisPNG:elixir-syntax-update

Conversation

@RisPNG

@RisPNG RisPNG commented Jul 10, 2025

Copy link
Copy Markdown
Contributor

We should show more love to Elixir syntax highlighting.

  • I updated the elixir-language-configuration.json to the latest one from ElixirLS as it includes more rules.
  • I added html-eex.json from ElixirLS as well and includes it in the package.json. I also made an update to the fileTypes in the html-eex.json since it is outdated. @josevalim himself made a recent change to the tmbundle syntaxes 6 months ago and I updated the fileTypes based on that.

Edit comment for more clarity.

@RisPNG RisPNG changed the title Elixir Syntax Update Elixir Syntax Highlighting Update Jul 11, 2025

@Blond11516 Blond11516 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the update!

Comment thread elixir-language-configuration.json
@Blond11516

Copy link
Copy Markdown
Collaborator

Hi @RisPNG! I just wanted to let you know I'm currently away on vacation and unfortunately did not manage to get back to this before leaving.

There's not much left to do here, but I will wait until I'm back to merge and release this. This way I can act quickly if this change were to break something. I dont expect it will but I'd rather be safe than sorry.

I should be able to look at this sometime soon after the 27th.

@RisPNG

RisPNG commented Jul 18, 2025

Copy link
Copy Markdown
Contributor Author

Hi @RisPNG! I just wanted to let you know I'm currently away on vacation and unfortunately did not manage to get back to this before leaving.

There's not much left to do here, but I will wait until I'm back to merge and release this. This way I can act quickly if this change were to break something. I dont expect it will but I'd rather be safe than sorry.

I should be able to look at this sometime soon after the 27th.

No worries, I'm currently using my own fork while waiting for the next update with this pr merged.

Enjoy your vacation!

@Blond11516
Blond11516 merged commit 00b6b32 into lexical-lsp:main Jul 30, 2025
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants