gocritic: set Go version in configuration

go.mod

@@ -32,7 +32,7 @@ require (
 	github.com/fatih/color v1.15.0
 	github.com/firefart/nonamedreturns v1.0.4
 	github.com/fzipp/gocyclo v0.6.0
-	github.com/go-critic/go-critic v0.7.0
+	github.com/go-critic/go-critic v0.8.0
 	github.com/go-xmlfmt/xmlfmt v1.1.2
 	github.com/gofrs/flock v0.8.1
 	github.com/golangci/check v0.0.0-20180506172741-cfe4005ccda2

pkg/config/linters_settings.go

@@ -371,6 +371,7 @@ type GoConstSettings struct {
 }
 
 type GoCriticSettings struct {
+	Go               string                           `mapstructure:"-"`
 	EnabledChecks    []string                         `mapstructure:"enabled-checks"`
 	DisabledChecks   []string                         `mapstructure:"disabled-checks"`
 	EnabledTags      []string                         `mapstructure:"enabled-tags"`

pkg/golinters/gocritic.go

@@ -113,6 +113,8 @@ func (w *goCriticWrapper) run(pass *analysis.Pass) ([]goanalysis.Issue, error) {
 
 	linterCtx := gocriticlinter.NewContext(pass.Fset, w.sizes)
 
+	linterCtx.SetGoVersion(w.settingsWrapper.Go)
+
 	enabledCheckers, err := w.buildEnabledCheckers(linterCtx)
 	if err != nil {
 		return nil, err

pkg/lint/lintersdb/manager.go

@@ -263,6 +263,10 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			govetCfg.Go = m.cfg.Run.Go
 		}
 
+		if gocriticCfg != nil {
+			gocriticCfg.Go = m.cfg.Run.Go
+		}
+
 		if gofumptCfg != nil && gofumptCfg.LangVersion == "" {
 			gofumptCfg.LangVersion = m.cfg.Run.Go
 		}

test/testdata/configs/gocritic.yml

@@ -1,10 +1,13 @@
+run:
+  go: 1.14
 linters-settings:
   gocritic:
     enabled-checks:
       - rangeValCopy
       - flagDeref
       - wrapperFunc
       - ruleguard
+      - syncMapLoadAndDelete # ignored because sync#Map.LoadAndDelete added in Go 1.15
     settings:
       rangeValCopy:
         sizeThreshold: 2

test/testdata/gocritic.go

@@ -6,6 +6,7 @@ import (
 	"flag"
 	"log"
 	"strings"
+	"sync"
 )
 
 var _ = *flag.Bool("global1", false, "") // want `flagDeref: immediate deref in \*flag.Bool\(.global1., false, ..\) is most likely an error; consider using flag\.BoolVar`
@@ -46,3 +47,13 @@ func gocriticDup(x bool) {
 func gocriticRuleWrapperFunc() {
 	strings.Replace("abcabc", "a", "d", -1) // want "ruleguard: this Replace call can be simplified.*"
 }
+
+func gocriticSink(args ...any) {}
+
+func gocriticIgnoreSyncMapLoadAndDelete(cond bool, m, m2 *sync.Map) {
+	actual, ok := m.Load("key")
+	if ok {
+		m.Delete("key")
+		gocriticSink(actual)
+	}
+}