dense-analysis / dense-analysis/ale

Deprecate `flow` linter

Open
#2,649 3 comments 1 reaction 0 assignees View on GitHub
old tool
Dominant language
Vim Script
Stars
14k
Forks
1.5k
Avg merge
17h 49m
Merged PRs (30d)
1

Description

Hi, I am from the Flow team and I am contemplating deprecating our non-LSP APIs. I wanted to discuss deprecating the `flow` ALE linter in favor of the `flow-language-server` linter.

I have to admit that I'm not a regular vim user, so my experience with ALE is very limited. But as far as I can tell, `flow-language-server` is a superset of `flow` since both handle diagnostics.

I think it'd be great if the two linters merged, so that `flow` used the LSP (perhaps only for newer Flow versions?) and `flow-language-server` got deprecated. `flow` is the better, more consistent name IMO.

If there are any issues with our LSP implementation that make people prefer the `flow` linter, I'd like to resolve those ASAP. The LSP is our primary API going forward.

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.