dense-analysis / dense-analysis/ale

List of supported tools places 💬 incorrectly, mistaking non-lsp:s for lsp:s and vice versa

Open
#5,024 0 comments 0 reactions 0 assignees View on GitHub
bug
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.