fix(hadolint): set code field as well (#3943)

This commit is contained in:
infokiller 2021-10-19 09:30:19 +03:00 committed by GitHub
parent 16898417e6
commit cb0f0e1d0b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 13 additions and 4 deletions

View File

@ -55,13 +55,19 @@ function! ale_linters#dockerfile#hadolint#Handle(buffer, lines) abort
let l:detail = 'hadolint could not parse the file because of a syntax error.' let l:detail = 'hadolint could not parse the file because of a syntax error.'
endif endif
call add(l:output, { let l:line_output = {
\ 'lnum': l:lnum, \ 'lnum': l:lnum,
\ 'col': l:colnum, \ 'col': l:colnum,
\ 'type': l:type, \ 'type': l:type,
\ 'text': l:text, \ 'text': l:text,
\ 'detail': l:detail \ 'detail': l:detail
\}) \}
if l:code isnot# ''
let l:line_output['code'] = l:code
endif
call add(l:output, l:line_output)
endfor endfor
return l:output return l:output

View File

@ -16,6 +16,7 @@ Execute(The hadolint handler should handle a normal example):
\ 'lnum': 1, \ 'lnum': 1,
\ 'col': 0, \ 'col': 0,
\ 'type': 'W', \ 'type': 'W',
\ 'code': 'DL3006',
\ 'text': "DL3006: Always tag the version of an image explicitly", \ 'text': "DL3006: Always tag the version of an image explicitly",
\ 'detail': "DL3006 ( https://github.com/hadolint/hadolint/wiki/DL3006 )\n\nAlways tag the version of an image explicitly", \ 'detail': "DL3006 ( https://github.com/hadolint/hadolint/wiki/DL3006 )\n\nAlways tag the version of an image explicitly",
\ }, \ },
@ -23,6 +24,7 @@ Execute(The hadolint handler should handle a normal example):
\ 'lnum': 4, \ 'lnum': 4,
\ 'col': 0, \ 'col': 0,
\ 'type': 'W', \ 'type': 'W',
\ 'code': 'DL3033',
\ 'text': "DL3033: Specify version with `yum install -y <package>-<version>`.", \ 'text': "DL3033: Specify version with `yum install -y <package>-<version>`.",
\ 'detail': "DL3033 ( https://github.com/hadolint/hadolint/wiki/DL3033 )\n\nSpecify version with `yum install -y <package>-<version>`.", \ 'detail': "DL3033 ( https://github.com/hadolint/hadolint/wiki/DL3033 )\n\nSpecify version with `yum install -y <package>-<version>`.",
\ }, \ },
@ -30,6 +32,7 @@ Execute(The hadolint handler should handle a normal example):
\ 'lnum': 12, \ 'lnum': 12,
\ 'col': 0, \ 'col': 0,
\ 'type': 'W', \ 'type': 'W',
\ 'code': 'SC2039',
\ 'text': "SC2039: In POSIX sh, brace expansion is undefined.", \ 'text': "SC2039: In POSIX sh, brace expansion is undefined.",
\ 'detail': "SC2039 ( https://github.com/koalaman/shellcheck/wiki/SC2039 )\n\nIn POSIX sh, brace expansion is undefined.", \ 'detail': "SC2039 ( https://github.com/koalaman/shellcheck/wiki/SC2039 )\n\nIn POSIX sh, brace expansion is undefined.",
\ }, \ },

View File

@ -68,14 +68,14 @@ Execute(command is correct when not docker):
Execute(test warnings from hadolint): Execute(test warnings from hadolint):
AssertEqual AssertEqual
\ [{'lnum': 10, 'col': 0, 'type': 'W', 'text': 'DL3007: Using latest is prone to errors', 'detail': "DL3007 ( https://github.com/hadolint/hadolint/wiki/DL3007 )\n\nUsing latest is prone to errors"}], \ [{'lnum': 10, 'col': 0, 'type': 'W', 'code': 'DL3007', 'text': 'DL3007: Using latest is prone to errors', 'detail': "DL3007 ( https://github.com/hadolint/hadolint/wiki/DL3007 )\n\nUsing latest is prone to errors"}],
\ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [ \ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [
\ '-:10 DL3007 warning: Using latest is prone to errors', \ '-:10 DL3007 warning: Using latest is prone to errors',
\ ]) \ ])
Execute(test warnings from shellcheck): Execute(test warnings from shellcheck):
AssertEqual AssertEqual
\ [{'lnum': 3, 'col': 0, 'type': 'W', 'text': 'SC2154: bar is referenced but not assigned.', 'detail': "SC2154 ( https://github.com/koalaman/shellcheck/wiki/SC2154 )\n\nbar is referenced but not assigned."}], \ [{'lnum': 3, 'col': 0, 'type': 'W', 'code': 'SC2154', 'text': 'SC2154: bar is referenced but not assigned.', 'detail': "SC2154 ( https://github.com/koalaman/shellcheck/wiki/SC2154 )\n\nbar is referenced but not assigned."}],
\ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [ \ ale_linters#dockerfile#hadolint#Handle(bufnr(''), [
\ '-:3 SC2154 warning: bar is referenced but not assigned.', \ '-:3 SC2154 warning: bar is referenced but not assigned.',
\ ]) \ ])