RepoMirrors/ale
1
0
Fork 0
mirror of https://github.com/dense-analysis/ale synced 2025-01-03 04:22:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added filename key for go build linter

Browse Source 
- Re: f224ce8a37

- 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.
This commit is contained in:
Jeff Willette 2017-11-20 23:50:14 +09:00
parent f20e5a4cf0
commit b9f02ffb27
2 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
ale_linters/go/gobuild.vim
View File

@ -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 " Description: go build for Go files
" inspired by work from dzhou121 <dzhou121@gmail.com> " inspired by work from dzhou121 <dzhou121@gmail.com>
@ -39,15 +40,12 @@ function! ale_linters#go#gobuild#GetMatches(lines) abort
endfunction endfunction
function! ale_linters#go#gobuild#Handler(buffer, lines) abort function! ale_linters#go#gobuild#Handler(buffer, lines) abort
let l:dir = expand('#' . a:buffer . ':p:h')
let l:output = [] let l:output = []
for l:match in ale_linters#go#gobuild#GetMatches(a:lines) 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, { call add(l:output, {
\ 'filename': ale#path#GetAbsPath(l:dir, l:match[1]),
\ 'lnum': l:match[2] + 0, \ 'lnum': l:match[2] + 0,
\ 'col': l:match[3] + 0, \ 'col': l:match[3] + 0,
\ 'text': l:match[4], \ 'text': l:match[4],

5
test/handler/test_gobuild_handler.vader
View File

@ -28,7 +28,7 @@ Execute (The gobuild handler should handle names with spaces):
\ ]), 'v:val[1:4]') \ ]), 'v:val[1:4]')
Execute (The gobuild handler should handle relative paths correctly): Execute (The gobuild handler should handle relative paths correctly):
silent file! /foo/bar/baz.go call ale#test#SetFilename('app/test.go')
AssertEqual AssertEqual
\ [ \ [
@ -37,8 +37,9 @@ Execute (The gobuild handler should handle relative paths correctly):
\ 'col': 0, \ 'col': 0,
\ 'text': 'missing argument for Printf("%s"): format reads arg 2, have only 1 args', \ 'text': 'missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ 'type': 'E', \ 'type': 'E',
\ 'filename': ale#path#Winify(expand('%:p:h') . '/test.go'),
\ }, \ },
\ ], \ ],
\ ale_linters#go#gobuild#Handler(bufnr(''), [ \ ale_linters#go#gobuild#Handler(bufnr(''), [
\ 'baz.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args', \ 'test.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ ]) \ ])