jshaver
/
ale
forked from github-mirrors/ale
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix #922 - Prefer nearer ESLint configuration files with lower precedence filenames for eslint --fix

This commit is contained in:
w0rp 2017-09-11 21:53:45 +01:00
parent 68bae8a1d1
commit 8a374a69a3
3 changed files with 20 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -2,19 +2,21 @@
" Description: Fixing files with eslint. " Description: Fixing files with eslint.
function! s:FindConfig(buffer) abort function! s:FindConfig(buffer) abort
for l:filename in [ for l:path in ale#path#Upwards(expand('#' . a:buffer . ':p:h'))
\ '.eslintrc.js', for l:basename in [
\ '.eslintrc.yaml', \ '.eslintrc.js',
\ '.eslintrc.yml', \ '.eslintrc.yaml',
\ '.eslintrc.json', \ '.eslintrc.yml',
\ '.eslintrc', \ '.eslintrc.json',
\ 'package.json', \ '.eslintrc',
\] \ 'package.json',
let l:config = ale#path#FindNearestFile(a:buffer, l:filename) \]
let l:config = ale#path#Simplify(l:path . '/' . l:basename)
if !empty(l:config) if filereadable(l:config)
return l:config return l:config
endif endif
endfor
endfor endfor
return '' return ''

0
test/eslint-test-files/react-app/subdir-with-config/.eslintrc Normal file
View File

14
test/fixers/test_eslint_fixer_callback.vader
View File

@ -17,18 +17,16 @@ Execute(The path to eslint.js should be run on Unix):
\ . ' --fix %t', \ . ' --fix %t',
\ }, \ },
\ ale#fixers#eslint#Fix(bufnr('')) \ ale#fixers#eslint#Fix(bufnr(''))
\
Execute(The lower priority configuration file in a nested directory should be preferred):
call ale#test#SetFilename('../eslint-test-files/react-app/subdir-with-config/testfile.js')
Execute(The eslint fixer with eslint.js should be run with node on Windows):
call ale#test#SetFilename('../eslint-test-files/react-app/subdir/testfile.js')
let g:ale_has_override['win32'] = 1
" We have to execute the file with node.
AssertEqual AssertEqual
\ { \ {
\ 'read_temporary_file': 1, \ 'read_temporary_file': 1,
\ 'command': ale#Escape('node.exe') . ' ' \ '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')) \ . ' --config ' . 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(''))