mirror of https://github.com/dense-analysis/ale
Merge pull request #761 from svanharmelen/f-improve-regex
Fixup #756 (gometalinter)
This commit is contained in:
commit
9e0d7a9d8c
|
@ -14,7 +14,7 @@ function! ale_linters#go#gometalinter#GetCommand(buffer) abort
|
|||
let l:options = ale#Var(a:buffer, 'go_gometalinter_options')
|
||||
|
||||
return ale#Escape(l:executable)
|
||||
\ . ' --include=' . ale#Escape(ale#util#EscapePCRE(l:filename))
|
||||
\ . ' --include=' . ale#Escape('^' . ale#util#EscapePCRE(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=' . ale#Escape(ale#util#EscapePCRE(expand('%')))
|
||||
\ . ' --include=' . ale#Escape('^' . ale#util#EscapePCRE(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=' . ale#Escape(ale#util#EscapePCRE(expand('%')))
|
||||
\ . ' --include=' . ale#Escape('^' . ale#util#EscapePCRE(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=' . ale#Escape(ale#util#EscapePCRE(expand('%')))
|
||||
\ . ' --include=' . ale#Escape('^' . ale#util#EscapePCRE(expand('%')))
|
||||
\ . ' --foobar'
|
||||
\ . ' ' . ale#Escape(getcwd()),
|
||||
\ ale_linters#go#gometalinter#GetCommand(bufnr(''))
|
||||
|
|
Loading…
Reference in New Issue