Merge pull request #756 from svanharmelen/b-gometalinter
Make gometalinter work again
This commit is contained in:
commit
a96cc92edf
@ -10,11 +10,11 @@ endfunction
|
||||
|
||||
function! ale_linters#go#gometalinter#GetCommand(buffer) abort
|
||||
let l:executable = ale_linters#go#gometalinter#GetExecutable(a:buffer)
|
||||
let l:filename = expand('#' . a:buffer . ':p')
|
||||
let l:filename = expand('#' . a:buffer)
|
||||
let l:options = ale#Var(a:buffer, 'go_gometalinter_options')
|
||||
|
||||
return ale#Escape(l:executable)
|
||||
\ . ' --include=''^' . l:filename . '.*$'''
|
||||
\ . ' --include=' . ale#Escape(l:filename)
|
||||
\ . (!empty(l:options) ? ' ' . l:options : '')
|
||||
\ . ' ' . ale#Escape(fnamemodify(l:filename, ':h'))
|
||||
endfunction
|
||||
|
@ -22,7 +22,7 @@ Execute(The gometalinter callback should return the right defaults):
|
||||
\ ale_linters#go#gometalinter#GetExecutable(bufnr(''))
|
||||
AssertEqual
|
||||
\ ale#Escape('gometalinter')
|
||||
\ . ' --include=''^' . expand('%:p') . '.*$'''
|
||||
\ . ' --include=' . ale#Escape(expand('%'))
|
||||
\ . ' ' . ale#Escape(getcwd()),
|
||||
\ ale_linters#go#gometalinter#GetCommand(bufnr(''))
|
||||
|
||||
@ -34,7 +34,7 @@ Execute(The gometalinter callback should use a configured executable):
|
||||
\ ale_linters#go#gometalinter#GetExecutable(bufnr(''))
|
||||
AssertEqual
|
||||
\ ale#Escape('something else')
|
||||
\ . ' --include=''^' . expand('%:p') . '.*$'''
|
||||
\ . ' --include=' . ale#Escape(expand('%'))
|
||||
\ . ' ' . ale#Escape(getcwd()),
|
||||
\ ale_linters#go#gometalinter#GetCommand(bufnr(''))
|
||||
|
||||
@ -43,7 +43,7 @@ Execute(The gometalinter callback should use configured options):
|
||||
|
||||
AssertEqual
|
||||
\ ale#Escape('gometalinter')
|
||||
\ . ' --include=''^' . expand('%:p') . '.*$'''
|
||||
\ . ' --include=' . ale#Escape(expand('%'))
|
||||
\ . ' --foobar'
|
||||
\ . ' ' . ale#Escape(getcwd()),
|
||||
\ ale_linters#go#gometalinter#GetCommand(bufnr(''))
|
||||
|
Loading…
Reference in New Issue
Block a user