VimPlug/ale
1
0
Fork 0
mirror of https://github.com/dense-analysis/ale.git synced 2025-12-07 21:24:33 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

remove --enable-all from golangci-lint options (#4488)

Browse Source 
* remove --enable-all from default golangci-lint options

* update golangci-lint options documentation

* update tests to use empty golangci-lint options
This commit is contained in:
Veselin Ivanov
2023-04-02 11:09:58 +02:00
committed by GitHub
parent 41e12fd640
commit b0ba31f88e
3 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/go/golangci_lint.vim
View File

@@ -1,7 +1,7 @@
" Author: Sascha Grunert <mail@saschagrunert.de> " Author: Sascha Grunert <mail@saschagrunert.de>
" Description: Adds support of golangci-lint " Description: Adds support of golangci-lint
call ale#Set('go_golangci_lint_options', '--enable-all') call ale#Set('go_golangci_lint_options', '')
call ale#Set('go_golangci_lint_executable', 'golangci-lint') call ale#Set('go_golangci_lint_executable', 'golangci-lint')
call ale#Set('go_golangci_lint_package', 0) call ale#Set('go_golangci_lint_package', 0)

2
doc/ale-go.txt
View File

@@ -120,7 +120,7 @@ g:ale_go_golangci_lint_executable *g:ale_go_golangci_lint_executable*
g:ale_go_golangci_lint_options *g:ale_go_golangci_lint_options* g:ale_go_golangci_lint_options *g:ale_go_golangci_lint_options*
*b:ale_go_golangci_lint_options* *b:ale_go_golangci_lint_options*
Type: |String| Type: |String|
Default: `'--enable-all'` Default: `''`
This variable can be changed to alter the command-line arguments to the This variable can be changed to alter the command-line arguments to the
golangci-lint invocation. golangci-lint invocation.

8
test/linter/test_golangci_lint.vader
View File

@@ -16,7 +16,7 @@ Execute(The golangci-lint defaults should be correct):
AssertLinter 'golangci-lint', AssertLinter 'golangci-lint',
\ ale#Escape('golangci-lint') \ ale#Escape('golangci-lint')
\ . ' run ' . ale#Escape(expand('%' . ':t')) \ . ' run ' . ale#Escape(expand('%' . ':t'))
\ . ' --enable-all' \ . ' '
Execute(The golangci-lint callback should use a configured executable): Execute(The golangci-lint callback should use a configured executable):
let b:ale_go_golangci_lint_executable = 'something else' let b:ale_go_golangci_lint_executable = 'something else'
@@ -24,7 +24,7 @@ Execute(The golangci-lint callback should use a configured executable):
AssertLinter 'something else', AssertLinter 'something else',
\ ale#Escape('something else') \ ale#Escape('something else')
\ . ' run ' . ale#Escape(expand('%' . ':t')) \ . ' run ' . ale#Escape(expand('%' . ':t'))
\ . ' --enable-all' \ . ' '
Execute(The golangci-lint callback should use configured options): Execute(The golangci-lint callback should use configured options):
let b:ale_go_golangci_lint_options = '--foobar' let b:ale_go_golangci_lint_options = '--foobar'
@@ -41,10 +41,10 @@ Execute(The golangci-lint callback should support environment variables):
\ ale#Env('GO111MODULE', 'on') \ ale#Env('GO111MODULE', 'on')
\ . ale#Escape('golangci-lint') \ . ale#Escape('golangci-lint')
\ . ' run ' . ale#Escape(expand('%' . ':t')) \ . ' run ' . ale#Escape(expand('%' . ':t'))
\ . ' --enable-all' \ . ' '
Execute(The golangci-lint `lint_package` option should use the correct command): Execute(The golangci-lint `lint_package` option should use the correct command):
let b:ale_go_golangci_lint_package = 1 let b:ale_go_golangci_lint_package = 1
AssertLinter 'golangci-lint', AssertLinter 'golangci-lint',
\ ale#Escape('golangci-lint') . ' run --enable-all' \ ale#Escape('golangci-lint') . ' run '