build(deps): bump github.com/go-critic/go-critic from 0.5.6 to 0.6.0 (#2041)

Co-authored-by: Fernandez Ludovic <[email protected]>

Author: dependabot[bot]

.github/workflows/codeql.yaml

@@ -26,6 +26,7 @@ jobs:
         # Supported options are ['csharp', 'cpp', 'go', 'java', 'javascript', 'python']
         # TODO: Enable for javascript later
         language: [ 'go' ]
+        golang: [ '1.16' ]
         # Learn more...
         # https://docs.github.com/en/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#overriding-automatic-language-detection
 
@@ -42,6 +43,11 @@ jobs:
       - run: git checkout HEAD^2
         if: ${{ github.event_name == 'pull_request' }}
 
+      - name: Install Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.golang }}
+
       # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
         uses: github/codeql-action/init@v1
@@ -56,4 +62,4 @@ jobs:
       - run: make build test
 
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@v1
+        uses: github/codeql-action/analyze@v1

.github/workflows/pr.yml

@@ -66,7 +66,6 @@ jobs:
     strategy:
       matrix:
         golang:
-          - 1.15
           - 1.16
           - 1.17
     steps:

.golangci.yml

@@ -50,8 +50,6 @@ linters-settings:
           - (github.com/golangci/golangci-lint/pkg/logutils.Log).Fatalf
   lll:
     line-length: 140
-  maligned:
-    suggest-new: true
   misspell:
     locale: US
   nolintlint:
@@ -117,6 +115,9 @@ linters:
   # - wsl
 
 issues:
+  exclude:
+    # disable this rule for go1.15 compatibility
+    - 'ioutilDeprecated:'
   # Excluding configuration per-path, per-linter, per-text and per-source
   exclude-rules:
     - path: _test\.go
@@ -141,7 +142,6 @@ issues:
       linters:
         - gomnd
 
-
 run:
   skip-dirs:
     - test/testdata_etc

go.mod

@@ -1,6 +1,6 @@
 module github.com/golangci/golangci-lint
 
-go 1.15
+go 1.16
 
 require (
 	4d63.com/gochecknoglobals v0.0.0-20201008074935-acfc0b28355a
@@ -21,7 +21,7 @@ require (
 	github.com/esimonov/ifshort v1.0.2
 	github.com/fatih/color v1.13.0
 	github.com/fzipp/gocyclo v0.3.1
-	github.com/go-critic/go-critic v0.5.6
+	github.com/go-critic/go-critic v0.6.1
 	github.com/go-xmlfmt/xmlfmt v0.0.0-20191208150333-d5b6f63a941b
 	github.com/gofrs/flock v0.8.1
 	github.com/golangci/check v0.0.0-20180506172741-cfe4005ccda2

go.sum

@@ -1,8 +1,8 @@
 [context.production.environment]
-  GO_VERSION = "1.15"
+  GO_VERSION = "1.16"
 
 [context.deploy-preview.environment]
-  GO_VERSION = "1.15"
+  GO_VERSION = "1.16"
 
 [context.branch-deploy.environment]
-  GO_VERSION = "1.15"
+  GO_VERSION = "1.16"

netlify.toml

@@ -492,6 +492,6 @@ func sizeOfReflectValueTreeBytes(rv reflect.Value, visitedPtrs map[uintptr]struc
 	case reflect.Invalid:
 		return 0
 	default:
-		panic("unknown rv of type " + fmt.Sprint(rv))
+		panic("unknown rv of type " + rv.String())
 	}
 }

pkg/golinters/goanalysis/runner_loadingpackage.go

@@ -34,7 +34,7 @@ func NewGocritic() *goanalysis.Linter {
 	}
 	return goanalysis.NewLinter(
 		gocriticName,
-		`Provides many diagnostics that check for bugs, performance and style issues.
+		`Provides diagnostics that check for bugs, performance and style issues.
 Extensible without recompilation through dynamic rules.
 Dynamic rules are written declaratively with AST patterns, filters, report message and optional suggestion.`,
 		[]*analysis.Analyzer{analyzer},

pkg/golinters/gocritic.go

@@ -92,7 +92,7 @@ func (i *Issue) Fingerprint() string {
 	}
 
 	hash := md5.New() //nolint:gosec
-	_, _ = hash.Write([]byte(fmt.Sprintf("%s%s%s", i.Pos.Filename, i.Text, firstLine)))
+	_, _ = fmt.Fprintf(hash, "%s%s%s", i.Pos.Filename, i.Text, firstLine)
 
 	return fmt.Sprintf("%X", hash.Sum(nil))
 }

pkg/result/issue.go

@@ -15,10 +15,10 @@ const noStagesText = "no stages"
 type Stopwatch struct {
 	name      string
 	startedAt time.Time
-	stages    map[string]time.Duration
 	log       logutils.Log
 
-	sync.Mutex
+	stages map[string]time.Duration
+	mu     sync.Mutex
 }
 
 func NewStopwatch(name string, log logutils.Log) *Stopwatch {
@@ -110,7 +110,7 @@ func (s *Stopwatch) TrackStage(name string, f func()) {
 	startedAt := time.Now()
 	f()
 
-	s.Lock()
+	s.mu.Lock()
 	s.stages[name] += time.Since(startedAt)
-	s.Unlock()
+	s.mu.Unlock()
 }