Skip to content

Commit 516a93f

Browse files
ldezSeigeC
authored andcommitted
dev: remove some TODO (golangci#2743)
* chore: use go1.18 to test the previous version of golangci-lint * chore: update gomnd configuration
1 parent d1c3f00 commit 516a93f

File tree

2 files changed

+14
-9
lines changed

2 files changed

+14
-9
lines changed

.github/workflows/pr.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,8 @@ jobs:
3434
- name: Install Go
3535
uses: actions/setup-go@v2
3636
with:
37-
go-version: 1.17 # TODO(ldez) the binary compiled with go1.17 doesn't work on go1.18
3837
# stable: 'false' # Keep this line to be able to use rc and beta version of Go (ex: 1.18.0-rc1).
39-
# go-version: ${{ env.GO_VERSION }} # TODO(ldez) the binary compiled with go1.17 doesn't work on go1.18
38+
go-version: ${{ env.GO_VERSION }}
4039
- name: lint
4140
uses: golangci/[email protected]
4241
with:

.golangci.yml

+13-7
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,19 @@ linters-settings:
3333
goimports:
3434
local-prefixes: github.com/golangci/golangci-lint
3535
gomnd:
36-
# TODO(ldez) must be rewritten after the v1.44.0 release.
37-
settings:
38-
mnd:
39-
# don't include the "operation" and "assign"
40-
checks: argument,case,condition,return
41-
ignored-numbers: 0,1,2,3
42-
ignored-functions: strings.SplitN
36+
# don't include the "operation" and "assign"
37+
checks:
38+
- argument
39+
- case
40+
- condition
41+
- return
42+
ignored-numbers:
43+
- '0'
44+
- '1'
45+
- '2'
46+
- '3'
47+
ignored-functions:
48+
- strings.SplitN
4349

4450
govet:
4551
check-shadowing: true

0 commit comments

Comments
 (0)