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

Remove style classification from E999 (#696)

Browse Source 
* Remove style classification from E999

* Update test_flake8_handler to reflect E999 changes
This commit is contained in:
Nick Krichevsky 2017-06-27 10:07:26 -04:00 committed by w0rp
parent 16ba9bd680
commit 14cca6d115
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/python/flake8.vim
View File

@ -136,7 +136,7 @@ function! ale_linters#python#flake8#Handle(buffer, lines) abort
\ 'type': 'W', \ 'type': 'W',
\} \}
if l:code[:0] ==# 'F' if l:code[:0] ==# 'F' || l:code ==# 'E999'
let l:item.type = 'E' let l:item.type = 'E'
elseif l:code[:0] ==# 'E' elseif l:code[:0] ==# 'E'
let l:item.type = 'E' let l:item.type = 'E'

9
test/handler/test_flake8_handler.vader
View File

@ -4,7 +4,7 @@ Before:
After: After:
call ale#linter#Reset() call ale#linter#Reset()
Execute(The flake8 handler should handle basic warnings): Execute(The flake8 handler should handle basic warnings and syntax errors):
AssertEqual AssertEqual
\ [ \ [
\ { \ {
@ -21,10 +21,17 @@ Execute(The flake8 handler should handle basic warnings):
\ 'text': 'W123: some warning', \ 'text': 'W123: some warning',
\ 'sub_type': 'style', \ 'sub_type': 'style',
\ }, \ },
\ {
\ 'lnum': 8,
\ 'col': 3,
\ 'type': 'E',
\ 'text': 'E999: SyntaxError: invalid syntax',
\ },
\ ], \ ],
\ ale_linters#python#flake8#Handle(1, [ \ ale_linters#python#flake8#Handle(1, [
\ 'stdin:6:6: E111 indentation is not a multiple of four', \ 'stdin:6:6: E111 indentation is not a multiple of four',
\ 'stdin:7:6: W123 some warning', \ 'stdin:7:6: W123 some warning',
\ 'stdin:8:3: E999 SyntaxError: invalid syntax',
\ ]) \ ])
Execute(The flake8 handler should set end column indexes should be set for certain errors): Execute(The flake8 handler should set end column indexes should be set for certain errors):