dense-analysis / dense-analysis/ale

A Few bugs with Metals integration

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

Description

I can submit a PR if people agree on what's required. Here are the issues:

1. `ale_linters/scala/metals.vim` tries to use a binary called `metals-vim`, but that's not what the binary is called (anymore at least.) It's just `metals`.
2. The root search code looks for `build.sc`, `build.sbt`, `.bloop`, and `.metals` using `ale#path#ResolveLocalPath`, but it looks like that function is only able to look for files, and `.bloop` and `.metals` are directories, so I think it's just the wrong function. What's the more correct function to use?

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.