chore: drop go-critic

because it panic at the start of golangci-lint and then don't allow to
check the go1.18 compatibility.

Author: ldez

pkg/config/linters_settings_gocritic.go

@@ -1,3 +1,6 @@
+//go:build go1.16 && !go1.18
+// +build go1.16,!go1.18
+
 package config
 
 import (

pkg/config/linters_settings_gocritic_go1_18.go

@@ -0,0 +1,32 @@
+//go:build go1.18
+// +build go1.18
+
+package config
+
+import (
+	"github.com/golangci/golangci-lint/pkg/logutils"
+)
+
+type GocriticCheckSettings map[string]interface{}
+
+type GocriticSettings struct {
+	EnabledChecks    []string                         `mapstructure:"enabled-checks"`
+	DisabledChecks   []string                         `mapstructure:"disabled-checks"`
+	EnabledTags      []string                         `mapstructure:"enabled-tags"`
+	DisabledTags     []string                         `mapstructure:"disabled-tags"`
+	SettingsPerCheck map[string]GocriticCheckSettings `mapstructure:"settings"`
+}
+
+func (s *GocriticSettings) InferEnabledChecks(_ logutils.Log) {}
+
+func (s *GocriticSettings) Validate(_ logutils.Log) error {
+	return nil
+}
+
+func (s *GocriticSettings) IsCheckEnabled(_ string) bool {
+	return false
+}
+
+func (s *GocriticSettings) GetLowercasedParams() map[string]GocriticCheckSettings {
+	return map[string]GocriticCheckSettings{}
+}

pkg/config/linters_settings_gocritic_test.go

@@ -1,3 +1,6 @@
+//go:build go1.16 && !go1.18
+// +build go1.16,!go1.18
+
 package config
 
 import (

pkg/golinters/gocritic.go

@@ -1,3 +1,6 @@
+//go:build go1.16 && !go1.18
+// +build go1.16,!go1.18
+
 package golinters
 
 import (

pkg/golinters/gocritic_go1_18.go

@@ -0,0 +1,35 @@
+//go:build go1.18
+// +build go1.18
+
+package golinters
+
+import (
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
+	"github.com/golangci/golangci-lint/pkg/lint/linter"
+)
+
+const gocriticName = "gocritic"
+
+func NewGocritic() *goanalysis.Linter {
+	analyzer := &analysis.Analyzer{
+		Name: gocriticName,
+		Doc:  goanalysis.TheOnlyanalyzerDoc,
+	}
+	return goanalysis.NewLinter(
+		gocriticName,
+		`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},
+		nil,
+	).WithContextSetter(func(lintCtx *linter.Context) {
+		analyzer.Run = func(pass *analysis.Pass) (interface{}, error) {
+			lintCtx.Log.Infof("FIXME") // FIXME
+			return nil, nil
+		}
+	}).WithIssuesReporter(func(*linter.Context) []goanalysis.Issue {
+		return nil
+	}).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}