mirror of
https://github.com/dense-analysis/ale
synced 2024-12-29 17:42:17 +00:00
18ec66bd21
I see no reason to do this? It is just setting the environment to what it already is? It was originally added in #297, but that entire PR is not a great idea in the first place; that PR (together with #270) tried to make the Go do non-standard and non-supported stuff like compiling packages outside of GOPATH. That's not something that works well (I tried), so was eventually removed in #465, but these "go env" calls remained, for no reason in particular, as far as I can think of. This will improve on #1834; you will now no longer get a confusing error (but still won't get a meaningful error; need to think how to do that).
20 lines
516 B
Plaintext
20 lines
516 B
Plaintext
Before:
|
|
call ale#assert#SetUpLinterTest('go', 'gobuild')
|
|
|
|
WithChainResults ['/foo/bar', '/foo/baz']
|
|
|
|
After:
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(The default commands should be correct):
|
|
AssertLinter 'go',
|
|
\ 'cd ' . ale#Escape(expand('%:p:h')) . ' && '
|
|
\ . 'go test -c -o /dev/null ./'
|
|
|
|
Execute(Extra options should be supported):
|
|
let g:ale_go_gobuild_options = '--foo-bar'
|
|
|
|
AssertLinter 'go',
|
|
\ 'cd ' . ale#Escape(expand('%:p:h')) . ' && '
|
|
\ . 'go test --foo-bar -c -o /dev/null ./'
|