chore: adds golangci config (#2174)

* removes rewrite-rules, sets gofmt.simplify to default true

* moves go vet step before the golangci-lint step

* fixes go vet issues with test files

* updates gocognit.min-complexity to default of 30

* removes deprecated options, primarily around run.go version
This commit is contained in:
CLoZengineer 2022-10-17 11:42:33 -04:00 committed by GitHub
parent 4e0a844cfb
commit 25dbbb2950
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 1857 additions and 5 deletions

View file

@ -37,13 +37,13 @@ jobs:
- run: go version
- name: go vet
run: go vet ./...
- name: golangci-lint
uses: golangci/golangci-lint-action@v3
with:
args: --timeout=5m
- name: go vet
run: go vet ./...
- name: test
run: go test ./...

1854
.golangci.yml Normal file

File diff suppressed because it is too large Load diff

View file

@ -12,7 +12,6 @@ import (
"github.com/featurebasedb/featurebase/v3/logger"
"github.com/featurebasedb/featurebase/v3/testhook"
"github.com/pkg/errors"
"github.com/stretchr/testify/assert"
"go.etcd.io/etcd/server/v3/embed"
"go.etcd.io/etcd/server/v3/etcdserver/api/v3client"

View file

@ -32,7 +32,6 @@ import (
"github.com/featurebasedb/featurebase/v3/test"
"github.com/featurebasedb/featurebase/v3/testhook"
. "github.com/featurebasedb/featurebase/v3/vprint" // nolint:staticcheck
. "github.com/featurebasedb/featurebase/v3/vprint" // nolint:staticcheck
"github.com/google/go-cmp/cmp"
"github.com/pkg/errors"
)