9fe7b1fe6a
Working directories are now set seperately from the commands so they can later be swapped out when running linters over projects is supported, and also better support filename mapping for running linters on other machines in future.
114 lines
3.7 KiB
VimL
114 lines
3.7 KiB
VimL
" Author: keith <k@keith.so>
|
|
" Description: pylint for python files
|
|
|
|
call ale#Set('python_pylint_executable', 'pylint')
|
|
call ale#Set('python_pylint_options', '')
|
|
call ale#Set('python_pylint_use_global', get(g:, 'ale_use_global_executables', 0))
|
|
call ale#Set('python_pylint_change_directory', 1)
|
|
call ale#Set('python_pylint_auto_pipenv', 0)
|
|
call ale#Set('python_pylint_use_msg_id', 0)
|
|
|
|
function! ale_linters#python#pylint#GetExecutable(buffer) abort
|
|
if (ale#Var(a:buffer, 'python_auto_pipenv') || ale#Var(a:buffer, 'python_pylint_auto_pipenv'))
|
|
\ && ale#python#PipenvPresent(a:buffer)
|
|
return 'pipenv'
|
|
endif
|
|
|
|
return ale#python#FindExecutable(a:buffer, 'python_pylint', ['pylint'])
|
|
endfunction
|
|
|
|
function! ale_linters#python#pylint#GetCwd(buffer) abort
|
|
if ale#Var(a:buffer, 'python_pylint_change_directory')
|
|
" pylint only checks for pylintrc in the packages above its current
|
|
" directory before falling back to user and global pylintrc.
|
|
" Run from project root, if found, otherwise buffer dir.
|
|
let l:project_root = ale#python#FindProjectRoot(a:buffer)
|
|
|
|
return !empty(l:project_root) ? l:project_root : '%s:h'
|
|
endif
|
|
|
|
return ''
|
|
endfunction
|
|
|
|
function! ale_linters#python#pylint#GetCommand(buffer, version) abort
|
|
let l:executable = ale_linters#python#pylint#GetExecutable(a:buffer)
|
|
let l:exec_args = l:executable =~? 'pipenv$'
|
|
\ ? ' run pylint'
|
|
\ : ''
|
|
|
|
return ale#Escape(l:executable) . l:exec_args
|
|
\ . ale#Pad(ale#Var(a:buffer, 'python_pylint_options'))
|
|
\ . ' --output-format text --msg-template="{path}:{line}:{column}: {msg_id} ({symbol}) {msg}" --reports n'
|
|
\ . (ale#semver#GTE(a:version, [2, 4, 0]) ? ' --from-stdin' : '')
|
|
\ . ' %s'
|
|
endfunction
|
|
|
|
function! ale_linters#python#pylint#Handle(buffer, lines) abort
|
|
let l:output = ale#python#HandleTraceback(a:lines, 10)
|
|
|
|
if !empty(l:output)
|
|
return l:output
|
|
endif
|
|
|
|
" Matches patterns like the following:
|
|
"
|
|
" test.py:4:4: W0101 (unreachable) Unreachable code
|
|
let l:pattern = '\v^[a-zA-Z]?:?[^:]+:(\d+):(\d+): ([[:alnum:]]+) \(([^(]*)\) (.*)$'
|
|
|
|
for l:match in ale#util#GetMatches(a:lines, l:pattern)
|
|
"let l:failed = append(0, l:match)
|
|
let l:code = l:match[3]
|
|
|
|
if (l:code is# 'C0303')
|
|
\ && !ale#Var(a:buffer, 'warn_about_trailing_whitespace')
|
|
" Skip warnings for trailing whitespace if the option is off.
|
|
continue
|
|
endif
|
|
|
|
if l:code is# 'I0011'
|
|
" Skip 'Locally disabling' message
|
|
continue
|
|
endif
|
|
|
|
if ale#Var(a:buffer, 'python_pylint_use_msg_id') is# 1
|
|
let l:code_out = l:code
|
|
else
|
|
let l:code_out = l:match[4]
|
|
endif
|
|
|
|
let l:item = {
|
|
\ 'lnum': l:match[1] + 0,
|
|
\ 'col': l:match[2] + 1,
|
|
\ 'text': l:match[5],
|
|
\ 'code': l:code_out,
|
|
\ 'type': 'W',
|
|
\}
|
|
|
|
if l:code[:0] is# 'E'
|
|
let l:item.type = 'E'
|
|
endif
|
|
|
|
call add(l:output, l:item)
|
|
endfor
|
|
|
|
return l:output
|
|
endfunction
|
|
|
|
call ale#linter#Define('python', {
|
|
\ 'name': 'pylint',
|
|
\ 'executable': function('ale_linters#python#pylint#GetExecutable'),
|
|
\ 'lint_file': {buffer -> ale#semver#RunWithVersionCheck(
|
|
\ buffer,
|
|
\ ale#Var(buffer, 'python_pylint_executable'),
|
|
\ '%e --version',
|
|
\ {buffer, version -> !ale#semver#GTE(version, [2, 4, 0])},
|
|
\ )},
|
|
\ 'cwd': function('ale_linters#python#pylint#GetCwd'),
|
|
\ 'command': {buffer -> ale#semver#RunWithVersionCheck(
|
|
\ buffer,
|
|
\ ale#Var(buffer, 'python_pylint_executable'),
|
|
\ '%e --version',
|
|
\ function('ale_linters#python#pylint#GetCommand'),
|
|
\ )},
|
|
\ 'callback': 'ale_linters#python#pylint#Handle',
|
|
\})
|