diff options
author | Jon Parise <jon@indelible.org> | 2017-02-09 19:15:49 -0800 |
---|---|---|
committer | Jon Parise <jon@indelible.org> | 2017-02-09 19:24:28 -0800 |
commit | d3e7d3d5e794dbce9802733a2bcc25e01f82b92b (patch) | |
tree | d65e31a96c7e2d57cbc0f373b1c8035aa74b4165 /ale_linters | |
parent | 5de445c041432b602c590a175809d89837cdf2b8 (diff) | |
download | ale-d3e7d3d5e794dbce9802733a2bcc25e01f82b92b.zip |
Pass the buffer's filename to Credo
By default, Credo attributes input from STDIN as though it came from a
file named `stdin`. This change passes the buffer's filename, too, so
that Credo can use that information when applying its configuration.
This is a nice improvement because files like `mix.exs` are normally
excluded from Credo-based linting. Previously, ALE would show lint
warnings for those files as they were edited. Now, they are correctly
honor the Credo configuration and don't produce lint output.
Diffstat (limited to 'ale_linters')
-rw-r--r-- | ale_linters/elixir/credo.vim | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ale_linters/elixir/credo.vim b/ale_linters/elixir/credo.vim index f79106d9..4eb5e4d0 100644 --- a/ale_linters/elixir/credo.vim +++ b/ale_linters/elixir/credo.vim @@ -3,8 +3,8 @@ function! ale_linters#elixir#credo#Handle(buffer, lines) abort " Matches patterns line the following: " - " stdin:19: F: Pipe chain should start with a raw value. - let l:pattern = '\v^stdin:(\d+):?(\d+)?: (.): (.+)$' + " lib/filename.ex:19:7: F: Pipe chain should start with a raw value. + let l:pattern = '\v^.+:(\d+):?(\d+)?: (.): (.+)$' let l:output = [] for l:line in a:lines @@ -41,5 +41,5 @@ endfunction call ale#linter#Define('elixir', { \ 'name': 'credo', \ 'executable': 'mix', - \ 'command': 'mix credo suggest --format=flycheck --read-from-stdin', + \ 'command': 'mix credo suggest --format=flycheck --read-from-stdin %s', \ 'callback': 'ale_linters#elixir#credo#Handle' }) |