hasufell
/
ale
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

#852 Capture error codes for shellcheck

This commit is contained in:
w0rp 2017-11-15 17:26:52 +00:00
parent b14377915b
commit 08f4f8f0fc
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ale_linters/sh/shellcheck.vim
View File

@ -71,7 +71,7 @@ function! ale_linters#sh#shellcheck#GetCommand(buffer, version_output) abort
endfunction
function! ale_linters#sh#shellcheck#Handle(buffer, lines) abort
let l:pattern = '\v^([a-zA-Z]?:?[^:]+):(\d+):(\d+)?:? ([^:]+): (.+)$'
let l:pattern = '\v^([a-zA-Z]?:?[^:]+):(\d+):(\d+)?:? ([^:]+): (.+) \[([^\]]+)\]$'
let l:output = []
for l:match in ale#util#GetMatches(a:lines, l:pattern)
@ -87,6 +87,7 @@ function! ale_linters#sh#shellcheck#Handle(buffer, lines) abort
\ 'lnum': str2nr(l:match[2]),
\ 'type': l:type,
\ 'text': l:match[5],
\ 'code': l:match[6],
\}
if !empty(l:match[3])

9
test/handler/test_shellcheck_handler.vader
View File

@ -11,13 +11,15 @@ Execute(The shellcheck handler should handle basic errors or warnings):
\ 'lnum': 2,
\ 'col': 1,
\ 'type': 'W',
\ 'text': 'In POSIX sh, ''let'' is not supported. [SC2039]',
\ 'text': 'In POSIX sh, ''let'' is not supported.',
\ 'code': 'SC2039',
\ },
\ {
\ 'lnum': 2,
\ 'col': 3,
\ 'type': 'E',
\ 'text': 'Don''t put spaces around the = in assignments. [SC1068]',
\ 'text': 'Don''t put spaces around the = in assignments.',
\ 'code': 'SC1068',
\ },
\ ],
\ ale_linters#sh#shellcheck#Handle(bufnr(''), [
@ -32,7 +34,8 @@ Execute(The shellcheck handler should handle notes):
\ 'lnum': 3,
\ 'col': 3,
\ 'type': 'I',
\ 'text': 'Double quote to prevent globbing and word splitting. [SC2086]',
\ 'text': 'Double quote to prevent globbing and word splitting.',
\ 'code': 'SC2086',
\ },
\ ],
\ ale_linters#sh#shellcheck#Handle(bufnr(''), [