We currently add //nolint comments for dot imports. See for example:
|
. "github.com/onsi/ginkgo/v2" //nolint:revive |
|
. "github.com/onsi/gomega" //nolint:revive |
At the same time we configure the linter to ignore these in .golangci.yml:
|
- linters: |
|
- revive |
|
- staticcheck |
|
path: (test)/.*.go |
There must be something wrong with this config or the //nolint comments would not be needed.
Figure out what the config should look like and remove the //nolint comments from all places where we use dot imports for ginkgo and gomega.
/help
/good-first-issue
We currently add
//nolintcomments for dot imports. See for example:cluster-api-provider-openstack/controllers/openstackcluster_controller_test.go
Lines 30 to 31 in 31797cd
At the same time we configure the linter to ignore these in .golangci.yml:
cluster-api-provider-openstack/.golangci.yml
Lines 163 to 166 in 31797cd
There must be something wrong with this config or the
//nolintcomments would not be needed.Figure out what the config should look like and remove the
//nolintcomments from all places where we use dot imports for ginkgo and gomega./help
/good-first-issue