diff options
| author | Katsuya Horiuchi <katsuya.horiuchi.biz@gmail.com> | 2019-05-10 01:28:18 +0900 |
|---|---|---|
| committer | w0rp <w0rp@users.noreply.github.com> | 2019-05-09 17:28:18 +0100 |
| commit | f444abdfe66696505cb891889b18c4144d12d4ea (patch) | |
| tree | c0b5c57c854c9e69dcc5a243fc2758f81aed98bc | |
| parent | 722c3e8dae80b9cea738ddb3df4f7d066d669e53 (diff) | |
| download | ale-f444abdfe66696505cb891889b18c4144d12d4ea.zip | |
Add option to show msg id when pylint is used (#2445)
* Add python_pylint_use_msg_id to tweak output of pylint
* Add test for ale_python_pylint_use_msg_id
* Add doc on ale_python_pylint_use_msg_id
| -rw-r--r-- | ale_linters/python/pylint.vim | 9 | ||||
| -rw-r--r-- | doc/ale-python.txt | 8 | ||||
| -rw-r--r-- | test/handler/test_pylint_handler.vader | 20 |
3 files changed, 36 insertions, 1 deletions
diff --git a/ale_linters/python/pylint.vim b/ale_linters/python/pylint.vim index 57e82691..b16d5355 100644 --- a/ale_linters/python/pylint.vim +++ b/ale_linters/python/pylint.vim @@ -6,6 +6,7 @@ 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')) @@ -64,11 +65,17 @@ function! ale_linters#python#pylint#Handle(buffer, lines) abort 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 + call add(l:output, { \ 'lnum': l:match[1] + 0, \ 'col': l:match[2] + 1, \ 'text': l:match[5], - \ 'code': l:match[4], + \ 'code': l:code_out, \ 'type': l:code[:0] is# 'E' ? 'E' : 'W', \}) endfor diff --git a/doc/ale-python.txt b/doc/ale-python.txt index dd946ad4..7dd3b65d 100644 --- a/doc/ale-python.txt +++ b/doc/ale-python.txt @@ -580,6 +580,14 @@ g:ale_python_pylint_auto_pipenv *g:ale_python_pylint_auto_pipenv* if true. This is overridden by a manually-set executable. +g:ale_python_pylint_use_msg_id *g:ale_python_pylint_use_msg_id* + *b:ale_python_pylint_use_msg_id* + Type: |Number| + Default: `0` + + Use message for output (e.g. I0011) instead of symbolic name of the message + (e.g. locally-disabled). + =============================================================================== pyls *ale-python-pyls* diff --git a/test/handler/test_pylint_handler.vader b/test/handler/test_pylint_handler.vader index 18f66526..ce7322f3 100644 --- a/test/handler/test_pylint_handler.vader +++ b/test/handler/test_pylint_handler.vader @@ -113,3 +113,23 @@ Execute(The pylint handler should parse Windows filenames): \ '------------------------------------------------------------------', \ 'Your code has been rated at 5.83/10 (previous run: 5.83/10, +0.00)', \ ]) + +Execute(Use msg_id): + let g:ale_python_pylint_use_msg_id = 1 + AssertEqual + \ [ + \ { + \ 'lnum': 13, + \ 'col': 6, + \ 'text': 'Undefined variable ''x''', + \ 'code': 'E0602', + \ 'type': 'E', + \ }, + \ ], + \ ale_linters#python#pylint#Handle(bufnr(''), [ + \ '************* Module test', + \ 'D:\acm\github\vim\tools\test.py:13:5: E0602 (undefined-variable) Undefined variable ''x''', + \ '', + \ '------------------------------------------------------------------', + \ 'Your code has been rated at 5.83/10 (previous run: 5.83/10, +0.00)', + \ ]) |