summaryrefslogtreecommitdiff
path: root/ale_linters/javascript/eslint.vim
diff options
context:
space:
mode:
authorw0rp <devw0rp@gmail.com>2017-06-20 10:50:38 +0100
committerw0rp <devw0rp@gmail.com>2017-06-20 10:50:38 +0100
commita105aa90a595ac5b8e2fe3f581a05bb705f5de21 (patch)
tree003b90a4bee11dc030cbbea7f2ad6a900107870c /ale_linters/javascript/eslint.vim
parentb96f5845ed7594cdc38355f6fae49a55a3725b2c (diff)
downloadale-a105aa90a595ac5b8e2fe3f581a05bb705f5de21.zip
Fix #668 - Support eslint for TypeScript
Diffstat (limited to 'ale_linters/javascript/eslint.vim')
-rw-r--r--ale_linters/javascript/eslint.vim87
1 files changed, 2 insertions, 85 deletions
diff --git a/ale_linters/javascript/eslint.vim b/ale_linters/javascript/eslint.vim
index 9f3bdcef..785b8bb6 100644
--- a/ale_linters/javascript/eslint.vim
+++ b/ale_linters/javascript/eslint.vim
@@ -1,92 +1,9 @@
" Author: w0rp <devw0rp@gmail.com>
" Description: eslint for JavaScript files
-let g:ale_javascript_eslint_options =
-\ get(g:, 'ale_javascript_eslint_options', '')
-
-function! ale_linters#javascript#eslint#GetCommand(buffer) abort
- let l:executable = ale#handlers#eslint#GetExecutable(a:buffer)
-
- if ale#Has('win32') && l:executable =~? 'eslint\.js$'
- " For Windows, if we detect an eslint.js script, we need to execute
- " it with node, or the file can be opened with a text editor.
- let l:head = 'node ' . ale#Escape(l:executable)
- else
- let l:head = ale#Escape(l:executable)
- endif
-
- let l:options = ale#Var(a:buffer, 'javascript_eslint_options')
-
- return l:head
- \ . (!empty(l:options) ? ' ' . l:options : '')
- \ . ' -f unix --stdin --stdin-filename %s'
-endfunction
-
-let s:col_end_patterns = [
-\ '\vParsing error: Unexpected token (.+) ',
-\ '\v''(.+)'' is not defined.',
-\ '\v%(Unexpected|Redundant use of) [''`](.+)[''`]',
-\ '\vUnexpected (console) statement',
-\]
-
-function! ale_linters#javascript#eslint#Handle(buffer, lines) abort
- let l:config_error_pattern = '\v^ESLint couldn''t find a configuration file'
- \ . '|^Cannot read config file'
- \ . '|^.*Configuration for rule .* is invalid'
-
- " Look for a message in the first few lines which indicates that
- " a configuration file couldn't be found.
- for l:line in a:lines[:10]
- if len(matchlist(l:line, l:config_error_pattern)) > 0
- return [{
- \ 'lnum': 1,
- \ 'text': 'eslint configuration error (type :ALEDetail for more information)',
- \ 'detail': join(a:lines, "\n"),
- \}]
- endif
- endfor
-
- " Matches patterns line the following:
- "
- " /path/to/some-filename.js:47:14: Missing trailing comma. [Warning/comma-dangle]
- " /path/to/some-filename.js:56:41: Missing semicolon. [Error/semi]
- let l:pattern = '^.*:\(\d\+\):\(\d\+\): \(.\+\) \[\(.\+\)\]$'
- " This second pattern matches lines like the following:
- "
- " /path/to/some-filename.js:13:3: Parsing error: Unexpected token
- let l:parsing_pattern = '^.*:\(\d\+\):\(\d\+\): \(.\+\)$'
- let l:output = []
-
- for l:match in ale#util#GetMatches(a:lines, [l:pattern, l:parsing_pattern])
- let l:type = 'Error'
- let l:text = l:match[3]
-
- " Take the error type from the output if available.
- if !empty(l:match[4])
- let l:type = split(l:match[4], '/')[0]
- let l:text .= ' [' . l:match[4] . ']'
- endif
-
- let l:obj = {
- \ 'lnum': l:match[1] + 0,
- \ 'col': l:match[2] + 0,
- \ 'text': l:text,
- \ 'type': l:type ==# 'Warning' ? 'W' : 'E',
- \}
-
- for l:col_match in ale#util#GetMatches(l:text, s:col_end_patterns)
- let l:obj.end_col = l:obj.col + len(l:col_match[1]) - 1
- endfor
-
- call add(l:output, l:obj)
- endfor
-
- return l:output
-endfunction
-
call ale#linter#Define('javascript', {
\ 'name': 'eslint',
\ 'executable_callback': 'ale#handlers#eslint#GetExecutable',
-\ 'command_callback': 'ale_linters#javascript#eslint#GetCommand',
-\ 'callback': 'ale_linters#javascript#eslint#Handle',
+\ 'command_callback': 'ale#handlers#eslint#GetCommand',
+\ 'callback': 'ale#handlers#eslint#Handle',
\})