From a6caa85a58d81aa4cd47172979a9c94aa8a902c6 Mon Sep 17 00:00:00 2001 From: Kevin Locke Date: Sun, 10 Feb 2019 20:37:13 -0700 Subject: pylama: Use %s instead of %t Although using %t to lint changes was desirable, many pylama checks use surrounding paths and file contents (e.g. C0103 module name, E0402 relative import beyond top, etc.) The more such errors I find during testing, the less %t seems like a good idea. Switch to %s. Also set `lint_file` to 1 and mark Pylama as a file linter in the docs. Signed-off-by: Kevin Locke --- ale_linters/python/pylama.vim | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'ale_linters') diff --git a/ale_linters/python/pylama.vim b/ale_linters/python/pylama.vim index 52b7c8c8..4ad25c0f 100644 --- a/ale_linters/python/pylama.vim +++ b/ale_linters/python/pylama.vim @@ -34,10 +34,13 @@ function! ale_linters#python#pylama#GetCommand(buffer) abort \ ? ' run pylama' \ : '' + " Note: Using %t to lint changes would be preferable, but many pylama + " checks use surrounding paths (e.g. C0103 module name, E0402 relative + " import beyond top, etc.). Neither is ideal. return l:cd_string \ . ale#Escape(l:executable) . l:exec_args \ . ale#Pad(ale#Var(a:buffer, 'python_pylama_options')) - \ . ' %t' + \ . ' %s' endfunction function! ale_linters#python#pylama#Handle(buffer, lines) abort @@ -67,12 +70,6 @@ function! ale_linters#python#pylama#Handle(buffer, lines) abort \} for l:match in ale#util#GetMatches(a:lines, l:pattern) - " Ignore C0103 for module name from temporary path (%t) which may not - " comply with module-rgx. - if l:match[3] is# 'C0103' && l:match[4] =~# '^Module name ' - continue - endif - call add(l:output, { \ 'lnum': str2nr(l:match[1]), \ 'col': str2nr(l:match[2]), @@ -91,4 +88,5 @@ call ale#linter#Define('python', { \ 'executable_callback': 'ale_linters#python#pylama#GetExecutable', \ 'command_callback': 'ale_linters#python#pylama#GetCommand', \ 'callback': 'ale_linters#python#pylama#Handle', +\ 'lint_file': 1, \}) -- cgit v1.2.3