hasufell/ale
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

#131 Keep the error types in the message for eslint

Browse Source
This commit is contained in:
w0rp 2016-10-20 18:07:45 +01:00
parent a34fb0a6a7
commit 7a06d276c2
2 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
ale_linters/javascript/eslint.vim
View File

@ -25,14 +25,8 @@ function! ale_linters#javascript#eslint#Handle(buffer, lines)
continue
endif
let l:text = l:match[3]
let l:marker = l:match[4]
let l:marker_parts = split(l:marker, '/')
let l:type = l:marker_parts[0]
if len(l:marker_parts) == 2
let l:text = l:text . ' (' . l:marker_parts[1] . ')'
endif
let l:type = split(l:match[4], '/')[0]
let l:text = l:match[3] . ' [' . l:match[4] . ']'
" vcol is Needed to indicate that the column is a character.
call add(l:output, {

2
test/test_linting_updates_loclist.vader
View File

@ -4,7 +4,7 @@ Given javascript (Some JavaScript with problems):
Before:
let g:ale_buffer_loclist_map = {}
let g:expected_data = {bufnr('%'): [{'lnum': 1, 'bufnr': bufnr('%'), 'vcol': 0, 'linter_name': 'eslint', 'nr': -1, 'type': 'W', 'col': 10, 'text': 'Infix operators must be spaced. (space-infix-ops)'}, {'lnum': 2, 'bufnr': bufnr('%'), 'vcol': 0, 'linter_name': 'eslint', 'nr': -1, 'type': 'E', 'col': 10, 'text': 'Missing semicolon. (semi)'}]}
let g:expected_data = {bufnr('%'): [{'lnum': 1, 'bufnr': bufnr('%'), 'vcol': 0, 'linter_name': 'eslint', 'nr': -1, 'type': 'W', 'col': 10, 'text': 'Infix operators must be spaced. [Warning/space-infix-ops]'}, {'lnum': 2, 'bufnr': bufnr('%'), 'vcol': 0, 'linter_name': 'eslint', 'nr': -1, 'type': 'E', 'col': 10, 'text': 'Missing semicolon. [Error/semi]'}]}
After:
let g:ale_buffer_loclist_map = {}