summaryrefslogtreecommitdiff
path: root/ale_linters/go
diff options
context:
space:
mode:
authorJeff Willette <jrwillette88@gmail.com>2017-11-20 23:50:14 +0900
committerJeff Willette <jrwillette88@gmail.com>2017-11-20 23:50:14 +0900
commitb9f02ffb275b1d120967ab163e79959d3673ffd6 (patch)
treedf475aab52a6f0199e36405023a04d3e8c05cb8a /ale_linters/go
parentf20e5a4cf0e56c3c89ef7be4730924be377e5f61 (diff)
downloadale-b9f02ffb275b1d120967ab163e79959d3673ffd6.zip
Added filename key for `go build` linter
- Re: f224ce8a377bbb3a0deb78b98fdc6c43555791e2 - The issues that prompted the above commit which reverted changes made to `go build` and `gometalinter` seemed to suggest that the main issue was with gometalinter and that changes should be put into different commits so they are independent of each other - This commit reinstates the changes to the `go build` linter which seem to be uncontested and it also seems absolutely necessary to show errors from all files in the package which may have caused a build failure.
Diffstat (limited to 'ale_linters/go')
-rw-r--r--ale_linters/go/gobuild.vim10
1 files changed, 4 insertions, 6 deletions
diff --git a/ale_linters/go/gobuild.vim b/ale_linters/go/gobuild.vim
index 143c2fd0..783b9e0a 100644
--- a/ale_linters/go/gobuild.vim
+++ b/ale_linters/go/gobuild.vim
@@ -1,4 +1,5 @@
-" Author: Joshua Rubin <joshua@rubixconsulting.com>, Ben Reedy <https://github.com/breed808>
+" Author: Joshua Rubin <joshua@rubixconsulting.com>, Ben Reedy <https://github.com/breed808>,
+" Jeff Willette <jrwillette88@gmail.com>
" Description: go build for Go files
" inspired by work from dzhou121 <dzhou121@gmail.com>
@@ -39,15 +40,12 @@ function! ale_linters#go#gobuild#GetMatches(lines) abort
endfunction
function! ale_linters#go#gobuild#Handler(buffer, lines) abort
+ let l:dir = expand('#' . a:buffer . ':p:h')
let l:output = []
for l:match in ale_linters#go#gobuild#GetMatches(a:lines)
- " Omit errors from imported go packages
- if !ale#path#IsBufferPath(a:buffer, l:match[1])
- continue
- endif
-
call add(l:output, {
+ \ 'filename': ale#path#GetAbsPath(l:dir, l:match[1]),
\ 'lnum': l:match[2] + 0,
\ 'col': l:match[3] + 0,
\ 'text': l:match[4],