Skip to content

Fix binary name for ALE config #103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Fix binary name for ALE config #103

wants to merge 1 commit into from

Conversation

bew
Copy link

@bew bew commented Apr 13, 2021

No description provided.

@lnicola
Copy link
Member

lnicola commented May 31, 2021

Are you sure about this? In the ALE docs I see that:

This plugin will look for linters in the ale_linters directory. Each directory within corresponds to a particular filetype in Vim, and each file in each directory corresponds to the name of a particular linter.

And the rust-analyzer linter is indeed called analyzer: https://github.com/dense-analysis/ale/blob/1b08791228f5aca4545a3fba6699b29a003028fe/ale_linters/rust/analyzer.vim.

PS: the manual is stored in the main rust-analyzer repo, not here.

@bew
Copy link
Author

bew commented Jun 1, 2021

Not sure at all, I don't personally use ale, but reading this page, all linter configs where using the name rust-analyser so I though it was a mistake 👀

@lnicola
Copy link
Member

lnicola commented Jun 1, 2021

Yeah, I think the current version is correct, at least according to the ALE docs. But thanks anyway!

@lnicola lnicola closed this Jun 1, 2021
lnicola added a commit that referenced this pull request Nov 15, 2021
lnicola added a commit that referenced this pull request Nov 15, 2021
lnicola added a commit that referenced this pull request Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants