Skip to content

Commit

Permalink
dev: remove some TODO (golangci#2743)
Browse files Browse the repository at this point in the history
* chore: use go1.18 to test the previous version of golangci-lint
* chore: update gomnd configuration
  • Loading branch information
ldez authored and SeigeC committed Apr 4, 2023
1 parent d1c3f00 commit 516a93f
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 9 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/pr.yml
Expand Up @@ -34,9 +34,8 @@ jobs:
- name: Install Go
uses: actions/setup-go@v2
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).
# 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
uses: golangci/golangci-lint-action@v3.1.0
with:
Expand Down
20 changes: 13 additions & 7 deletions .golangci.yml
Expand Up @@ -33,13 +33,19 @@ linters-settings:
goimports:
local-prefixes: github.com/golangci/golangci-lint
gomnd:
# TODO(ldez) must be rewritten after the v1.44.0 release.
settings:
mnd:
# don't include the "operation" and "assign"
checks: argument,case,condition,return
ignored-numbers: 0,1,2,3
ignored-functions: strings.SplitN
# don't include the "operation" and "assign"
checks:
- argument
- case
- condition
- return
ignored-numbers:
- '0'
- '1'
- '2'
- '3'
ignored-functions:
- strings.SplitN

govet:
check-shadowing: true
Expand Down

0 comments on commit 516a93f

Please sign in to comment.