Merge pull request #1052 from natw/tflint-no-stdin

don't use stdin with tflint
This commit is contained in:
w0rp 2017-10-26 20:25:55 +01:00 committed by GitHub
commit 3ab069c9bd
3 changed files with 4 additions and 4 deletions

View File

@ -46,7 +46,7 @@ function! ale_linters#terraform#tflint#GetCommand(buffer) abort
let l:cmd .= ' ' . l:opts let l:cmd .= ' ' . l:opts
endif endif
let l:cmd .= ' -f json' let l:cmd .= ' -f json %t'
return l:cmd return l:cmd
endfunction endfunction

View File

@ -21,12 +21,12 @@ Execute(The default executable should be configurable):
Execute(The default command should be good): Execute(The default command should be good):
let g:ale_terraform_tflint_executable = 'tflint' let g:ale_terraform_tflint_executable = 'tflint'
AssertEqual AssertEqual
\ ale#Escape('tflint') . ' -f json', \ ale#Escape('tflint') . ' -f json %t',
\ ale_linters#terraform#tflint#GetCommand(bufnr('')) \ ale_linters#terraform#tflint#GetCommand(bufnr(''))
Execute(Overriding things should work): Execute(Overriding things should work):
let g:ale_terraform_tflint_executable = 'fnord' let g:ale_terraform_tflint_executable = 'fnord'
let g:ale_terraform_tflint_options = '--whatever' let g:ale_terraform_tflint_options = '--whatever'
AssertEqual AssertEqual
\ ale#Escape('fnord') . ' --whatever -f json', \ ale#Escape('fnord') . ' --whatever -f json %t',
\ ale_linters#terraform#tflint#GetCommand(bufnr('')) \ ale_linters#terraform#tflint#GetCommand(bufnr(''))

View File

@ -13,6 +13,6 @@ Execute(adjacent config file should be found):
\ ale#Escape('tflint') \ ale#Escape('tflint')
\ . ' --config ' \ . ' --config '
\ . ale#Escape(ale#path#Winify(g:dir . '/tflint-test-files/foo/.tflint.hcl')) \ . ale#Escape(ale#path#Winify(g:dir . '/tflint-test-files/foo/.tflint.hcl'))
\ . ' -f json' \ . ' -f json %t'
\ ), \ ),
\ ale_linters#terraform#tflint#GetCommand(bufnr('')) \ ale_linters#terraform#tflint#GetCommand(bufnr(''))