mirror of
https://github.com/dense-analysis/ale.git
synced 2025-12-06 04:34:25 +08:00
no need to write and read temporary file with syntax_tree (#4834)
Some checks failed
CI / build_image (push) Has been cancelled
CI / test_ale (--linters-only) (push) Has been cancelled
CI / test_ale (--neovim-06-only) (push) Has been cancelled
CI / test_ale (--neovim-08-only) (push) Has been cancelled
CI / test_ale (--vim-80-only) (push) Has been cancelled
CI / test_ale (--vim-90-only) (push) Has been cancelled
Some checks failed
CI / build_image (push) Has been cancelled
CI / test_ale (--linters-only) (push) Has been cancelled
CI / test_ale (--neovim-06-only) (push) Has been cancelled
CI / test_ale (--neovim-08-only) (push) Has been cancelled
CI / test_ale (--vim-80-only) (push) Has been cancelled
CI / test_ale (--vim-90-only) (push) Has been cancelled
This commit is contained in:
@@ -6,7 +6,7 @@ function! ale#fixers#syntax_tree#GetCommand(buffer) abort
|
||||
let l:options = ale#Var(a:buffer, 'ruby_syntax_tree_options')
|
||||
|
||||
return ale#ruby#EscapeExecutable(l:executable, 'stree')
|
||||
\ . ' write'
|
||||
\ . ' format'
|
||||
\ . (!empty(l:options) ? ' ' . l:options : '')
|
||||
\ . ' %t'
|
||||
endfunction
|
||||
@@ -14,6 +14,5 @@ endfunction
|
||||
function! ale#fixers#syntax_tree#Fix(buffer) abort
|
||||
return {
|
||||
\ 'command': ale#fixers#syntax_tree#GetCommand(a:buffer),
|
||||
\ 'read_temporary_file': 1,
|
||||
\}
|
||||
endfunction
|
||||
|
||||
@@ -18,9 +18,8 @@ Execute(The syntax_tree callback should return the correct default values):
|
||||
|
||||
AssertEqual
|
||||
\ {
|
||||
\ 'read_temporary_file': 1,
|
||||
\ 'command': ale#Escape(g:ale_ruby_syntax_tree_executable)
|
||||
\ . ' write %t',
|
||||
\ . ' format %t',
|
||||
\ },
|
||||
\ ale#fixers#syntax_tree#Fix(bufnr(''))
|
||||
|
||||
@@ -30,8 +29,7 @@ Execute(The syntax_tree callback should include custom options):
|
||||
|
||||
AssertEqual
|
||||
\ {
|
||||
\ 'read_temporary_file': 1,
|
||||
\ 'command': ale#Escape(g:ale_ruby_syntax_tree_executable)
|
||||
\ . ' write --print-width=100 --plugins=plugin/trailing_comma %t',
|
||||
\ . ' format --print-width=100 --plugins=plugin/trailing_comma %t',
|
||||
\ },
|
||||
\ ale#fixers#syntax_tree#Fix(bufnr(''))
|
||||
|
||||
Reference in New Issue
Block a user