mirror of
https://github.com/dense-analysis/ale
synced 2024-12-17 20:05:31 +00:00
24a937e04f
Checking whole packages instead of individual files is more sensible default for golang projects. Without this we get many cryptic `typecheck` errors in ALE that do not show when running in terminal or CI.
48 lines
1.3 KiB
Plaintext
48 lines
1.3 KiB
Plaintext
Before:
|
|
Save g:ale_go_go111module
|
|
|
|
call ale#assert#SetUpLinterTest('go', 'golangci_lint')
|
|
call ale#test#SetFilename('test.go')
|
|
|
|
After:
|
|
Restore
|
|
|
|
unlet! b:ale_go_go111module
|
|
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(The golangci-lint defaults should be correct):
|
|
AssertLinterCwd '%s:h',
|
|
AssertLinter 'golangci-lint',
|
|
\ ale#Escape('golangci-lint') . ' run '
|
|
|
|
Execute(The golangci-lint callback should use a configured executable):
|
|
let b:ale_go_golangci_lint_executable = 'something else'
|
|
|
|
AssertLinter 'something else',
|
|
\ ale#Escape('something else')
|
|
\ . ' run '
|
|
|
|
Execute(The golangci-lint callback should use configured options):
|
|
let b:ale_go_golangci_lint_options = '--foobar'
|
|
|
|
AssertLinter 'golangci-lint',
|
|
\ ale#Escape('golangci-lint')
|
|
\ . ' run '
|
|
\ . '--foobar'
|
|
|
|
Execute(The golangci-lint callback should support environment variables):
|
|
let b:ale_go_go111module = 'on'
|
|
|
|
AssertLinter 'golangci-lint',
|
|
\ ale#Env('GO111MODULE', 'on')
|
|
\ . ale#Escape('golangci-lint')
|
|
\ . ' run '
|
|
|
|
Execute(The golangci-lint `lint_package` option should use the correct command):
|
|
let b:ale_go_golangci_lint_package = 0
|
|
AssertLinter 'golangci-lint',
|
|
\ ale#Escape('golangci-lint')
|
|
\ . ' run ' . ale#Escape(expand('%' . ':t'))
|
|
\ . ' '
|