go-critic/ruleguard: compatibility with Go 1.18

[pierrre]

Welcome

• Yes, I'm using a binary release within 2 latest major releases. Only such installations are supported.
• Yes, I've searched similar issues on GitHub and didn't find any.
• Yes, I've included all information below (version, config, etc).
• Yes, I've tried with the standalone linter if available. (https://golangci-lint.run/usage/linters/)

Description of the problem

Go 1.18beta1 has been released.

When I run golangci-lint run I get:

panic: load embedded ruleguard rules: rules/rules.go:13: can't load fmt

goroutine 1 [running]:
github.com/go-critic/go-critic/checkers.init.9()
	github.com/go-critic/[email protected]/checkers/checkers.go:58 +0x4b4

Version of golangci-lint

$ golangci-lint --version
panic: load embedded ruleguard rules: rules/rules.go:13: can't load fmt

goroutine 1 [running]:
github.com/go-critic/go-critic/checkers.init.9()
	github.com/go-critic/[email protected]/checkers/checkers.go:58 +0x4b4

But I'm using v1.43, installed with curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.43.0.

Configuration file

$ cat .golangci.yml

I'm not using any config file

Go environment

$ go version && go env
go version go1.18beta1 linux/amd64
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/pierre/.cache/go-build"
GOENV="/home/pierre/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/pierre/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/pierre/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/pierre/.gimme/versions/go1.18beta1.src"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/pierre/.gimme/versions/go1.18beta1.src/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18beta1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2019785836=/tmp/go-build -gno-record-gcc-switches"

Verbose output of running

$ golangci-lint cache clean
$ golangci-lint run -v
panic: load embedded ruleguard rules: rules/rules.go:13: can't load fmt

goroutine 1 [running]:
github.com/go-critic/go-critic/checkers.init.9()
	github.com/go-critic/[email protected]/checkers/checkers.go:58 +0x4b4

Code example or link to a public repository

// add your code here

[ldez]

It's more an issue with go-critic/ruleguard than golangci-lint.

I recommend opening an issue on the go-critic repo.

[pierrre]

Very strange, I've got exactly the same error messages as
go-critic/go-critic#1157
#2374

[Goldziher]

Well, I am using 1.18beta2, and this is still an issue. Furthermore, I cannot bypass it by disabling go-ciritc in the golangci-lint configs, because it seems its always loaded no matter what. This is my config:

linters-settings:
  goimports:
    local-prefixes: github.com/StreamElements/
  govet:
    check-shadowing: true
linters:
  disable:
    - gocritic
  enable:
    - bidichk
    - deadcode
    - durationcheck
    - errcheck
    - errorlint
    - exportloopref
    - goimports
    - goconst
    # - gocritic
    - godot
    - godox
    - goheader
    - gomoddirectives
    - gomodguard
    - goprintffuncname
    - gosec
    - ineffassign
    - gosimple
    - govet
    - nolintlint
    - predeclared
    - rowserrcheck
    - sqlclosecheck
    - staticcheck
    - structcheck
    - tenv
    - testpackage
    - typecheck
    - unconvert
    - unparam
    - unused
    - varcheck
    - wastedassign
    - whitespace
issues:
  fix: true

And this is what I am getting on the command line (running using pre-commit):

panic: load embedded ruleguard rules: rules/rules.go:13: can't load fmt

goroutine 1 [running]:
github.com/go-critic/go-critic/checkers.init.22()
	/Users/naamanhirschfeld/.cache/pre-commit/reposlmx3ef2/golangenv-default/pkg/mod/github.com/go-critic/[email protected]/checkers/embedded_rules.go:46 +0x52c

[ldez]

Take a look at #2438

[Goldziher]

would be lovely to see this released :)

[EwenQuim]

Furthermore, I cannot bypass it by disabling go-critic in the golangci-lint configs, because it seems its always loaded no matter what.

Same issue here, it really bugs me as I am not even using go-critic...
Thanks for #2438, will be watching it closely! :)

[tehsphinx]

Since go1.18 is released now, can we up the prio on this? Maybe tag it with Bug?

[ldez]

Take a look at #2438.

Yes go1.18 is released but the tooling provided by Go is not ready, it's not our fault.

I will close this issue in favor of the PR.

[atombender]

The issue should stay open until PR is merged.

[ldez]

The issue should stay open until PR is merged.

This issue is only about ruleguard but the problem is global.

The PR provides information and details for all the linters, not only one.

We can open a global issue but we will say the same things as in the PR.

[atombender]

A single tracking issue for Go 1.18 would be great. There may need to be multiple PRs, and PRs typically have too much chatter about the actual work.

[ldez]

There may need to be multiple PRs, and PRs typically have too much chatter about the actual work.

I don't think so because the problem is related to only one thing: the SSA analyzer.

If you want to help, you can contribute to solving this issue golang/go#48525

[ldez]

I will open an issue because I feel like we will have to repeat too much time the same thing on different issues.

[ldez]

#2649