Run docstring lint in seprate job
License: MIT Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>
This commit is contained in:
parent
4df601ea6a
commit
eae1851aaf
@ -98,13 +98,18 @@ jobs:
|
|||||||
default: go/circleci-golang
|
default: go/circleci-golang
|
||||||
golangci-lint-version:
|
golangci-lint-version:
|
||||||
type: string
|
type: string
|
||||||
default: 1.15.0
|
default: 1.17.1
|
||||||
concurrency:
|
concurrency:
|
||||||
type: string
|
type: string
|
||||||
default: '2'
|
default: '2'
|
||||||
description: |
|
description: |
|
||||||
Concurrency used to run linters. Defaults to 2 because NumCPU is not
|
Concurrency used to run linters. Defaults to 2 because NumCPU is not
|
||||||
aware of container CPU limits.
|
aware of container CPU limits.
|
||||||
|
args:
|
||||||
|
type: string
|
||||||
|
default: ''
|
||||||
|
description: |
|
||||||
|
Arguments to pass to golangci-lint
|
||||||
executor: << parameters.executor >>
|
executor: << parameters.executor >>
|
||||||
steps:
|
steps:
|
||||||
- install-deps
|
- install-deps
|
||||||
@ -118,15 +123,16 @@ jobs:
|
|||||||
name: Lint
|
name: Lint
|
||||||
command: |
|
command: |
|
||||||
golangci-lint run -v \
|
golangci-lint run -v \
|
||||||
--concurrency << parameters.concurrency >>
|
--concurrency << parameters.concurrency >> << parameters.args >>
|
||||||
|
|
||||||
|
|
||||||
workflows:
|
workflows:
|
||||||
version: 2
|
version: 2
|
||||||
ci:
|
ci:
|
||||||
jobs:
|
jobs:
|
||||||
|
- lint
|
||||||
- lint:
|
- lint:
|
||||||
golangci-lint-version: 1.17.1
|
args: "--no-config --exclude-use-default=false --disable-all --enable golint"
|
||||||
- test:
|
- test:
|
||||||
codecov-upload: true
|
codecov-upload: true
|
||||||
- mod-tidy-check
|
- mod-tidy-check
|
||||||
|
@ -20,6 +20,7 @@ issues:
|
|||||||
exclude:
|
exclude:
|
||||||
- "func name will be used as test\\.Test.* by other packages, and that stutters; consider calling this"
|
- "func name will be used as test\\.Test.* by other packages, and that stutters; consider calling this"
|
||||||
- "Potential file inclusion via variable"
|
- "Potential file inclusion via variable"
|
||||||
|
- "should have comment or be unexported"
|
||||||
|
|
||||||
exclude-use-default: false
|
exclude-use-default: false
|
||||||
exclude-rules:
|
exclude-rules:
|
||||||
|
Loading…
Reference in New Issue
Block a user