summaryrefslogtreecommitdiff
path: root/ale_linters/go
diff options
context:
space:
mode:
authorSander van Harmelen <sander@xanzy.io>2018-02-15 12:23:36 +0100
committerSander van Harmelen <sander@xanzy.io>2018-02-20 16:11:35 +0100
commitab5257c3442f5d5b5236905a4c77f4f09a24d8b5 (patch)
tree0b9bd088252e6c15f4991d52d6a1155d4040ad19 /ale_linters/go
parentfcb7932d7d61cda142da7597c9df4da4847f0ca8 (diff)
downloadale-ab5257c3442f5d5b5236905a4c77f4f09a24d8b5.zip
This fixes issue #936 by linting the whole package
Diffstat (limited to 'ale_linters/go')
-rw-r--r--ale_linters/go/staticcheck.vim24
1 files changed, 23 insertions, 1 deletions
diff --git a/ale_linters/go/staticcheck.vim b/ale_linters/go/staticcheck.vim
index 255fd17c..ce9e6e38 100644
--- a/ale_linters/go/staticcheck.vim
+++ b/ale_linters/go/staticcheck.vim
@@ -1,10 +1,32 @@
" Author: Ben Reedy <https://github.com/breed808>
" Description: staticcheck for Go files
+call ale#Set('go_staticcheck_options', '')
+call ale#Set('go_staticcheck_lint_package', 0)
+
+function! ale_linters#go#staticcheck#GetCommand(buffer) abort
+ let l:filename = expand('#' . a:buffer . ':t')
+ let l:options = ale#Var(a:buffer, 'go_staticcheck_options')
+ let l:lint_package = ale#Var(a:buffer, 'go_staticcheck_lint_package')
+
+ " BufferCdString is used so that we can be sure the paths output from
+ " staticcheck can be calculated to absolute paths in the Handler
+ if l:lint_package
+ return ale#path#BufferCdString(a:buffer)
+ \ . 'staticcheck'
+ \ . (!empty(l:options) ? ' ' . l:options : '') . ' .'
+ endif
+
+ return ale#path#BufferCdString(a:buffer)
+ \ . 'staticcheck'
+ \ . (!empty(l:options) ? ' ' . l:options : '')
+ \ . ' ' . ale#Escape(l:filename)
+endfunction
+
call ale#linter#Define('go', {
\ 'name': 'staticcheck',
\ 'executable': 'staticcheck',
-\ 'command': 'staticcheck %s',
+\ 'command_callback': 'ale_linters#go#staticcheck#GetCommand',
\ 'callback': 'ale#handlers#unix#HandleAsWarning',
\ 'output_stream': 'both',
\ 'lint_file': 1,