diff options
| author | Dmitri Vereshchagin <dmitri.vereshchagin@gmail.com> | 2021-11-12 10:50:56 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-11-12 16:50:56 +0900 |
| commit | 76c2293e68a6cad3b192062743d25b8daa082205 (patch) | |
| tree | 140877fefa091bdf7935e3b4731c0af1c6cd16c9 | |
| parent | ff26ed72312838acbc801cc5e410faa5bab181e1 (diff) | |
| download | ale-76c2293e68a6cad3b192062743d25b8daa082205.zip | |
Add sub_type to Elvis output (#3976)
Elvis is Erlang style reviewer, thus all loclist objects should have
sub_type set to style.
| -rw-r--r-- | ale_linters/erlang/elvis.vim | 1 | ||||
| -rw-r--r-- | test/handler/test_erlang_elvis_handler.vader | 3 |
2 files changed, 4 insertions, 0 deletions
diff --git a/ale_linters/erlang/elvis.vim b/ale_linters/erlang/elvis.vim index 31dea3dd..0fb85c07 100644 --- a/ale_linters/erlang/elvis.vim +++ b/ale_linters/erlang/elvis.vim @@ -12,6 +12,7 @@ function! ale_linters#erlang#elvis#Handle(buffer, lines) abort \ 'lnum': str2nr(l:match[1]), \ 'text': s:AbbreviateMessage(l:match[2]), \ 'type': 'W', + \ 'sub_type': 'style', \}) endfor diff --git a/test/handler/test_erlang_elvis_handler.vader b/test/handler/test_erlang_elvis_handler.vader index 365376c8..0a6d756d 100644 --- a/test/handler/test_erlang_elvis_handler.vader +++ b/test/handler/test_erlang_elvis_handler.vader @@ -11,11 +11,13 @@ Execute(Warning messages should be handled): \ 'lnum': 11, \ 'text': "Replace the 'if' expression on line 11 with a 'case' expression or function clauses.", \ 'type': 'W', + \ 'sub_type': 'style', \ }, \ { \ 'lnum': 20, \ 'text': 'Remove the debug call to io:format/1 on line 20.', \ 'type': 'W', + \ 'sub_type': 'style', \ }, \ ], \ ale_linters#erlang#elvis#Handle(bufnr(''), [ @@ -30,6 +32,7 @@ Execute(Line length message shouldn't contain the line itself): \ 'lnum': 24, \ 'text': 'Line 24 is too long.', \ 'type': 'W', + \ 'sub_type': 'style', \ }, \ ], \ ale_linters#erlang#elvis#Handle(bufnr(''), [ |