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

Don't use temporary file for rustfmt fixer

Browse Source 
rustfmt normally acts on a file in place, and applies configuration
from rustfmt.toml files according to the path of the file.

Using a temporary file for rustfmt breaks this functionality, so
removing the '%t' from the rustfmt command.
This commit is contained in:
Björn Lindström 2017-12-27 23:03:46 +07:00
parent c165c7c5d1
commit b0eaddadc9
2 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
autoload/ale/fixers/rustfmt.vim
View File

@ -10,8 +10,6 @@ function! ale#fixers#rustfmt#Fix(buffer) abort
return {
\ 'command': ale#Escape(l:executable)
\ . (empty(l:options) ? '' : ' ' . l:options)
\ . ' %t',
\ 'read_temporary_file': 1,
\ . (empty(l:options) ? '' : ' ' . l:options),
\}
endfunction

8
test/fixers/test_rustfmt_fixer_callback.vader
View File

@ -18,9 +18,7 @@ Execute(The rustfmt callback should return the correct default values):
AssertEqual
\ {
\ 'read_temporary_file': 1,
\ 'command': ale#Escape('xxxinvalid')
\ . ' %t',
\ 'command': ale#Escape('xxxinvalid'),
\ },
\ ale#fixers#rustfmt#Fix(bufnr(''))
@ -30,9 +28,7 @@ Execute(The rustfmt callback should include custom rustfmt options):
AssertEqual
\ {
\ 'read_temporary_file': 1,
\ 'command': ale#Escape('xxxinvalid')
\ . ' ' . g:ale_rust_rustfmt_options
\ . ' %t',
\ . ' ' . g:ale_rust_rustfmt_options,
\ },
\ ale#fixers#rustfmt#Fix(bufnr(''))