go1.21 support (golangci#3922)

Author: ldez

.github/workflows/documentation.yml

@@ -11,7 +11,7 @@ jobs:
     name: Build and deploy documentation
     runs-on: ubuntu-latest
     env:
-      GO_VERSION: '1.20'
+      GO_VERSION: '1.21'
       NODE_VERSION: '20.x'
       CGO_ENABLED: 0
 

.github/workflows/post-release.yml

@@ -18,7 +18,7 @@ jobs:
           # ex:
           # - 1.18beta1 -> 1.18.0-beta.1
           # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: '1.20'
+          go-version: '1.21'
 
       - name: Update GitHub action config
         run: make assets/github-action-config.json

.github/workflows/pr-extra.yml

@@ -17,7 +17,7 @@ jobs:
           # ex:
           # - 1.18beta1 -> 1.18.0-beta.1
           # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: '1.20'
+          go-version: '1.21'
       - name: Run go list
         run: go list -json -m all > go.list
       - name: Nancy

.github/workflows/pr.yml

@@ -6,7 +6,7 @@ on:
   pull_request:
 
 env:
-  GO_VERSION: '1.20'
+  GO_VERSION: '1.21'
 
 jobs:
   # Check if there is any dirty change for go mod tidy
@@ -41,7 +41,9 @@ jobs:
           # ex:
           # - 1.18beta1 -> 1.18.0-beta.1
           # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: ${{ env.GO_VERSION }}
+          # TODO(ldez) must be changed after the first release of golangci-lint with go1.21
+          # go-version: ${{ env.GO_VERSION }}
+          go-version: '1.20'
       - name: lint
         uses: golangci/golangci-lint-action@v3.6.0
         with:
@@ -88,8 +90,8 @@ jobs:
     strategy:
       matrix:
         golang:
-          - 1.19
           - '1.20'
+          - '1.21'
     steps:
       - uses: actions/checkout@v3
       - name: Install Go

.github/workflows/tag.yml

@@ -18,7 +18,7 @@ jobs:
           # ex:
           # - 1.18beta1 -> 1.18.0-beta.1
           # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: '1.20'
+          go-version: '1.21'
       - name: Unshallow
         run: git fetch --prune --unshallow
 
@@ -48,7 +48,7 @@ jobs:
           # ex:
           # - 1.18beta1 -> 1.18.0-beta.1
           # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: '1.20'
+          go-version: '1.21'
 
       - name: Unshallow
         run: git fetch --prune --unshallow

.golangci.yml

@@ -155,6 +155,8 @@ issues:
       text: "SA1019: (.+).(GoVersion|LangVersion) is deprecated: use the global `run.go` instead."
     - path: pkg/golinters/unused.go
       text: "rangeValCopy: each iteration copies 160 bytes \\(consider pointers or indexing\\)"
+    - path: test/(fix|linters)_test.go
+      text: "string `gocritic.go` has 3 occurrences, make it a constant"
 
 run:
   timeout: 5m

build/Dockerfile

@@ -1,5 +1,5 @@
 # stage 1 building the code
-FROM golang:1.20 as builder
+FROM golang:1.21 as builder
 
 ARG VERSION
 ARG SHORT_COMMIT
@@ -10,7 +10,7 @@ WORKDIR /golangci
 RUN CGO_ENABLED=0 go build -trimpath -ldflags "-s -w -X main.version=$VERSION -X main.commit=$SHORT_COMMIT -X main.date=$DATE" -o golangci-lint ./cmd/golangci-lint/main.go
 
 # stage 2
-FROM golang:1.20
+FROM golang:1.21
 # related to https://github.com/golangci/golangci-lint/issues/3107
 ENV GOROOT /usr/local/go
 # don't place it into $GOPATH/bin because Drone mounts $GOPATH as volume

build/alpine.Dockerfile

@@ -1,5 +1,5 @@
 # stage 1 building the code
-FROM golang:1.20-alpine as builder
+FROM golang:1.21-alpine as builder
 
 ARG VERSION
 ARG SHORT_COMMIT
@@ -15,7 +15,7 @@ RUN apk --no-cache add gcc musl-dev git mercurial
 RUN CGO_ENABLED=0 go build -trimpath -ldflags "-s -w -X main.version=$VERSION -X main.commit=$SHORT_COMMIT -X main.date=$DATE" -o golangci-lint ./cmd/golangci-lint/main.go
 
 # stage 2
-FROM golang:1.20-alpine
+FROM golang:1.21-alpine
 # related to https://github.com/golangci/golangci-lint/issues/3107
 ENV GOROOT /usr/local/go
 # gcc is required to support cgo;

go.mod

@@ -1,6 +1,6 @@
 module github.com/golangci/golangci-lint
 
-go 1.19
+go 1.20
 
 require (
 	4d63.com/gocheckcompilerdirectives v1.2.1

pkg/config/config.go

@@ -41,13 +41,13 @@ type Version struct {
 	Debug  bool   `mapstructure:"debug"`
 }
 
-func IsGreaterThanOrEqualGo118(v string) bool {
+func IsGreaterThanOrEqualGo121(v string) bool {
 	v1, err := hcversion.NewVersion(strings.TrimPrefix(v, "go"))
 	if err != nil {
 		return false
 	}
 
-	limit, err := hcversion.NewVersion("1.18")
+	limit, err := hcversion.NewVersion("1.21")
 	if err != nil {
 		return false
 	}