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

There seems to be a bug in eslint that causes the --config option to

Browse Source 
not detect node_modules correctly. The `-c` option, however, works fine.
This commit is contained in:
Ray Zane 2017-09-24 13:57:18 -04:00
parent 2bd352370f
commit cb56cbb714
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
autoload/ale/fixers/eslint.vim
View File

@ -11,7 +11,7 @@ function! ale#fixers#eslint#Fix(buffer) abort
return { return {
\ 'command': ale#node#Executable(a:buffer, l:executable) \ 'command': ale#node#Executable(a:buffer, l:executable)
\ . ' --config ' . ale#Escape(l:config) \ . ' -c ' . ale#Escape(l:config)
\ . ' --fix %t', \ . ' --fix %t',
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\} \}

8
test/fixers/test_eslint_fixer_callback.vader
View File

@ -13,7 +13,7 @@ Execute(The path to eslint.js should be run on Unix):
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\ 'command': \ 'command':
\ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js')) \ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js'))
\ . ' --config ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/.eslintrc.js')) \ . ' -c ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/.eslintrc.js'))
\ . ' --fix %t', \ . ' --fix %t',
\ }, \ },
\ ale#fixers#eslint#Fix(bufnr('')) \ ale#fixers#eslint#Fix(bufnr(''))
@ -26,7 +26,7 @@ Execute(The lower priority configuration file in a nested directory should be pr
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\ 'command': \ 'command':
\ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js')) \ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js'))
\ . ' --config ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/subdir-with-config/.eslintrc')) \ . ' -c ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/subdir-with-config/.eslintrc'))
\ . ' --fix %t', \ . ' --fix %t',
\ }, \ },
\ ale#fixers#eslint#Fix(bufnr('')) \ ale#fixers#eslint#Fix(bufnr(''))
@ -39,7 +39,7 @@ Execute(package.json should be used as a last resort):
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\ 'command': \ 'command':
\ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js')) \ ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/node_modules/eslint/bin/eslint.js'))
\ . ' --config ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/.eslintrc.js')) \ . ' -c ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/react-app/.eslintrc.js'))
\ . ' --fix %t', \ . ' --fix %t',
\ }, \ },
\ ale#fixers#eslint#Fix(bufnr('')) \ ale#fixers#eslint#Fix(bufnr(''))
@ -51,7 +51,7 @@ Execute(package.json should be used as a last resort):
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\ 'command': \ 'command':
\ ale#Escape(simplify(g:dir . '/../eslint-test-files/node_modules/.bin/eslint')) \ ale#Escape(simplify(g:dir . '/../eslint-test-files/node_modules/.bin/eslint'))
\ . ' --config ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/package.json')) \ . ' -c ' . ale#Escape(simplify(g:dir . '/../eslint-test-files/package.json'))
\ . ' --fix %t', \ . ' --fix %t',
\ }, \ },
\ ale#fixers#eslint#Fix(bufnr('')) \ ale#fixers#eslint#Fix(bufnr(''))