dense-analysis / dense-analysis/ale
List of supported tools places 💬 incorrectly, mistaking non-lsp:s for lsp:s and vice versa
- Dominant language
- Vim Script
- Stars
- 14k
- Forks
- 1.5k
- Avg merge
- 17h 49m
- Merged PRs (30d)
- 1
Description
## Information
**VIM version** N/A, Operating System: N/A
ALE commit ac691b0.
## What went wrong
It seems pull request #5011 was merged prematurely without the outstanding issue in it being addressed.
As is written in it:
> I see that rust-analyzer is missing its 💬 and that cargo is misclassified to have one.
There is also this statement which I still hold for true:
> Unassisted by technology, it likely becomes impossibly to keep the list correct as new tools are added.
I was waiting for @F-Kearney to have a go at running the check-lsp-emojis script.
Now when the buggy version got merged, I'm not sure what to do. Maybe you @w0rp are keen on running the suggested script and reclassify? What's your position on running something like it in the test suite to keep the list of tools accurately classified?
Contributor guide
Assessment
This issue has not been assessed yet.