dev: remove some TODO (#2743)
* chore: use go1.18 to test the previous version of golangci-lint * chore: update gomnd configuration
This commit is contained in:
parent
e4945f75cc
commit
bffc8cefe2
3
.github/workflows/pr.yml
vendored
3
.github/workflows/pr.yml
vendored
@ -34,9 +34,8 @@ jobs:
|
|||||||
- name: Install Go
|
- name: Install Go
|
||||||
uses: actions/setup-go@v2
|
uses: actions/setup-go@v2
|
||||||
with:
|
with:
|
||||||
go-version: 1.17 # TODO(ldez) the binary compiled with go1.17 doesn't work on go1.18
|
|
||||||
# stable: 'false' # Keep this line to be able to use rc and beta version of Go (ex: 1.18.0-rc1).
|
# stable: 'false' # Keep this line to be able to use rc and beta version of Go (ex: 1.18.0-rc1).
|
||||||
# go-version: ${{ env.GO_VERSION }} # TODO(ldez) the binary compiled with go1.17 doesn't work on go1.18
|
go-version: ${{ env.GO_VERSION }}
|
||||||
- name: lint
|
- name: lint
|
||||||
uses: golangci/golangci-lint-action@v3.1.0
|
uses: golangci/golangci-lint-action@v3.1.0
|
||||||
with:
|
with:
|
||||||
|
@ -33,13 +33,19 @@ linters-settings:
|
|||||||
goimports:
|
goimports:
|
||||||
local-prefixes: github.com/golangci/golangci-lint
|
local-prefixes: github.com/golangci/golangci-lint
|
||||||
gomnd:
|
gomnd:
|
||||||
# TODO(ldez) must be rewritten after the v1.44.0 release.
|
|
||||||
settings:
|
|
||||||
mnd:
|
|
||||||
# don't include the "operation" and "assign"
|
# don't include the "operation" and "assign"
|
||||||
checks: argument,case,condition,return
|
checks:
|
||||||
ignored-numbers: 0,1,2,3
|
- argument
|
||||||
ignored-functions: strings.SplitN
|
- case
|
||||||
|
- condition
|
||||||
|
- return
|
||||||
|
ignored-numbers:
|
||||||
|
- '0'
|
||||||
|
- '1'
|
||||||
|
- '2'
|
||||||
|
- '3'
|
||||||
|
ignored-functions:
|
||||||
|
- strings.SplitN
|
||||||
|
|
||||||
govet:
|
govet:
|
||||||
check-shadowing: true
|
check-shadowing: true
|
||||||
|
Loading…
x
Reference in New Issue
Block a user