From 056fbd08f653dbf7a66d4e73f7b68646c697583f Mon Sep 17 00:00:00 2001 From: Henrik Farre Date: Sat, 31 Jul 2021 05:35:43 +0200 Subject: [PATCH] Do not use tempfiles for ansible lint (#3846) * Used %s instead of std in * Set lint_file to 1 for ansible-lint so it does not use temp files * Fix test for ansible-lint --- ale_linters/ansible/ansible_lint.vim | 3 ++- test/linter/test_ansible_lint.vader | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ale_linters/ansible/ansible_lint.vim b/ale_linters/ansible/ansible_lint.vim index e1b123ab..3b443369 100644 --- a/ale_linters/ansible/ansible_lint.vim +++ b/ale_linters/ansible/ansible_lint.vim @@ -73,7 +73,7 @@ endfunction function! ale_linters#ansible#ansible_lint#GetCommand(buffer, version) abort let l:commands = { - \ '>=5.0.0': '%e --nocolor --parseable-severity -x yaml -', + \ '>=5.0.0': '%e --nocolor --parseable-severity -x yaml %s', \ '<5.0.0': '%e --nocolor -p %t' \} let l:command = ale#semver#GTE(a:version, [5, 0]) ? l:commands['>=5.0.0'] : l:commands['<5.0.0'] @@ -91,6 +91,7 @@ call ale#linter#Define('ansible', { \ '%e --version', \ function('ale_linters#ansible#ansible_lint#GetCommand'), \ )}, +\ 'lint_file': 1, \ 'callback': {buffer, lines -> ale#semver#RunWithVersionCheck( \ buffer, \ ale_linters#ansible#ansible_lint#GetExecutable(buffer), diff --git a/test/linter/test_ansible_lint.vader b/test/linter/test_ansible_lint.vader index a35d8604..6fb8acb5 100644 --- a/test/linter/test_ansible_lint.vader +++ b/test/linter/test_ansible_lint.vader @@ -13,7 +13,7 @@ Execute(The ansible_lint version <5.0.0 command callback should return default s Execute(The ansible_lint version >=5.0.0 command callback should return default string): GivenCommandOutput ['v5.1.2'] - AssertLinter 'ansible-lint', ale#Escape('ansible-lint') . ' --nocolor --parseable-severity -x yaml -' + AssertLinter 'ansible-lint', ale#Escape('ansible-lint') . ' --nocolor --parseable-severity -x yaml %s' Execute(The ansible_lint executable should be configurable): let g:ale_ansible_ansible_lint_executable = '~/.local/bin/ansible-lint'