RepoMirrors
/
ale
mirror of https://github.com/dense-analysis/ale
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use stdin/stdout to communicate with gofmt

This commit is contained in:
Andrey Popp 2020-10-03 17:57:59 +03:00
parent b91c6c2edd
commit 96ec33e6d6
2 changed files with 3 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
autoload/ale/fixers/gofmt.vim
View File

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

14
test/fixers/test_gofmt_fixer_callback.vader
View File

@ -21,10 +21,7 @@ Execute(The gofmt callback should return the correct default values):
AssertEqual AssertEqual
\ { \ {
\ 'read_temporary_file': 1, \ 'command': ale#Escape('xxxinvalid'),
\ 'command': ale#Escape('xxxinvalid')
\ . ' -l -w'
\ . ' %t',
\ }, \ },
\ ale#fixers#gofmt#Fix(bufnr('')) \ ale#fixers#gofmt#Fix(bufnr(''))
@ -35,11 +32,8 @@ Execute(The gofmt callback should include custom gofmt options):
AssertEqual AssertEqual
\ { \ {
\ 'read_temporary_file': 1,
\ 'command': ale#Escape('xxxinvalid') \ 'command': ale#Escape('xxxinvalid')
\ . ' -l -w' \ . ' ' . g:ale_go_gofmt_options,
\ . ' ' . g:ale_go_gofmt_options
\ . ' %t',
\ }, \ },
\ ale#fixers#gofmt#Fix(bufnr('')) \ ale#fixers#gofmt#Fix(bufnr(''))
@ -50,9 +44,7 @@ Execute(The gofmt callback should support Go environment variables):
AssertEqual AssertEqual
\ { \ {
\ 'read_temporary_file': 1,
\ 'command': ale#Env('GO111MODULE', 'off') \ 'command': ale#Env('GO111MODULE', 'off')
\ . ale#Escape('xxxinvalid') . ' -l -w' \ . ale#Escape('xxxinvalid')
\ . ' %t',
\ }, \ },
\ ale#fixers#gofmt#Fix(bufnr('')) \ ale#fixers#gofmt#Fix(bufnr(''))