2016-09-15 16:09:04 -04:00
|
|
|
" Author: Chris Kyrouac - https://github.com/fijshion
|
2016-10-03 14:55:55 -04:00
|
|
|
" Description: jscs for JavaScript files
|
2016-09-15 16:09:04 -04:00
|
|
|
|
2017-07-08 13:37:21 -04:00
|
|
|
call ale#Set('javascript_jscs_executable', 'jscs')
|
2018-04-09 14:11:20 -04:00
|
|
|
call ale#Set('javascript_jscs_use_global', get(g:, 'ale_use_global_executables', 0))
|
2017-07-08 13:37:21 -04:00
|
|
|
|
|
|
|
function! ale_linters#javascript#jscs#GetExecutable(buffer) abort
|
|
|
|
return ale#node#FindExecutable(a:buffer, 'javascript_jscs', [
|
|
|
|
\ 'node_modules/.bin/jscs',
|
|
|
|
\])
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale_linters#javascript#jscs#GetCommand(buffer) abort
|
|
|
|
" Search for a local JShint config locaation, and default to a global one.
|
|
|
|
let l:jscs_config = ale#path#ResolveLocalPath(
|
|
|
|
\ a:buffer,
|
|
|
|
\ '.jscsrc',
|
|
|
|
\ get(g:, 'ale_jscs_config_loc', '')
|
|
|
|
\)
|
|
|
|
|
|
|
|
let l:command = ale#Escape(ale_linters#javascript#jscs#GetExecutable(a:buffer))
|
|
|
|
let l:command .= ' --reporter inline --no-colors'
|
|
|
|
|
|
|
|
if !empty(l:jscs_config)
|
|
|
|
let l:command .= ' --config ' . ale#Escape(l:jscs_config)
|
|
|
|
endif
|
|
|
|
|
|
|
|
let l:command .= ' -'
|
|
|
|
|
|
|
|
return l:command
|
|
|
|
endfunction
|
|
|
|
|
|
|
|
function! ale_linters#javascript#jscs#Handle(buffer, lines) abort
|
|
|
|
" Matches patterns looking like the following
|
|
|
|
"
|
|
|
|
" foobar.js: line 2, col 1, Expected indentation of 1 characters
|
|
|
|
"
|
2017-11-18 18:55:47 -05:00
|
|
|
let l:pattern = '\v^.*:\s+line (\d+),\s+col\s+(\d+),\s+(.*)$'
|
2017-07-08 13:37:21 -04:00
|
|
|
let l:output = []
|
|
|
|
|
2017-11-18 18:55:47 -05:00
|
|
|
for l:match in ale#util#GetMatches(a:lines, l:pattern)
|
2017-07-08 13:37:21 -04:00
|
|
|
let l:obj = {
|
|
|
|
\ 'lnum': l:match[1] + 0,
|
|
|
|
\ 'col': l:match[2] + 0,
|
|
|
|
\ 'text': l:match[3]
|
|
|
|
\}
|
|
|
|
|
2017-11-18 18:55:47 -05:00
|
|
|
let l:code_match = matchlist(l:match[3], '\v([^ :]+): (.+)$')
|
|
|
|
|
|
|
|
if !empty(l:code_match)
|
|
|
|
let l:obj.code = l:code_match[1]
|
|
|
|
let l:obj.text = l:code_match[2]
|
|
|
|
endif
|
|
|
|
|
2017-07-08 13:37:21 -04:00
|
|
|
call add(l:output, l:obj)
|
|
|
|
endfor
|
|
|
|
|
|
|
|
return l:output
|
|
|
|
endfunction
|
|
|
|
|
First pass at optimizing ale to autoload (#80)
* First pass at optimizing ale to autoload
First off, the structure/function names should be revised a bit,
but I will wait for @w0rp's input before unifying the naming style.
Second off, the docs probably need some more work, I just did some
simple find-and-replace work.
With that said, this pull brings major performance gains for ale. On my
slowest system, fully loading ale and all its code takes around 150ms.
I have moved all of ale's autoload-able code to autoload/, and in
addition, implemented lazy-loading of linters. This brings load time on
that same system down to 5ms.
The only downside of lazy loading is that `g:ale_linters` cannot be
changed at runtime; however, it also speeds up performance at runtime by
simplfying the logic greatly.
Please let me know what you think!
Closes #59
* Address Travis/Vint errors
For some reason, ale isn't running vint for me...
* Incorporate feedback, make fixes
Lazy-loading logic is much improved.
* Add header comments; remove incorrect workaround
* Remove unneeded plugin guards
* Fix lazy-loading linter logic
Set the wrong variable....
* Fix capitialization
2016-10-10 14:51:29 -04:00
|
|
|
call ale#linter#Define('javascript', {
|
2016-09-15 16:09:04 -04:00
|
|
|
\ 'name': 'jscs',
|
2017-07-08 13:37:21 -04:00
|
|
|
\ 'executable_callback': 'ale_linters#javascript#jscs#GetExecutable',
|
|
|
|
\ 'command_callback': 'ale_linters#javascript#jscs#GetCommand',
|
|
|
|
\ 'callback': 'ale_linters#javascript#jscs#Handle',
|
2016-09-14 15:22:29 -04:00
|
|
|
\})
|
2017-07-08 13:37:21 -04:00
|
|
|
|