Skip to content

Add support for glint LSP (#4651)#5077

Open
sukima wants to merge 1 commit intodense-analysis:masterfrom
sukima:add-glint-lsp-support
Open

Add support for glint LSP (#4651)#5077
sukima wants to merge 1 commit intodense-analysis:masterfrom
sukima:add-glint-lsp-support

Conversation

@sukima
Copy link

@sukima sukima commented Dec 19, 2025

Closes #4651

Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Please update the documentation accordingly:

  • Add entry to doc/ale-html.txt. There is no Glimmer section so this should be added too.
  • Add entry to doc/ale-supported-languages-and-tools.txt. Similarly a new Glimmer section may need to be added.
  • Add entry to supported-tools.md
  • Create new doc/ale-gts.txt and doc/ale-gjs.txt files and add an entry describing the new LSP configuration variables (e.g. glimmer_glint_executable). You can add the entry to one (e.g. ale-gts.txt) and then simply reference it in the other. See doc/ale-asciidoc.txt for an example.
@sukima sukima force-pushed the add-glint-lsp-support branch 2 times, most recently from f7036b6 to 67f983b Compare December 23, 2025 14:46
@sukima sukima requested a review from hsanson December 23, 2025 14:46
@sukima
Copy link
Author

sukima commented Jan 5, 2026

  • Add support for global executable use.
@sukima sukima force-pushed the add-glint-lsp-support branch from 67f983b to c68f54d Compare February 25, 2026 20:13
@sukima
Copy link
Author

sukima commented Mar 3, 2026

Glint V2 has moved into a tsserver plugin and thus this linter might not be necessary?

Further experimentation is needed. Looking for any insight from other Vim & Ember users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants