Skip to content

Commit

Permalink
Merge pull request #2504 from EladLeev/golangci-cleanup
Browse files Browse the repository at this point in the history
chore(ci): bump golangci version, cleanup, depguard config
  • Loading branch information
dnwe committed Jul 17, 2023
2 parents 5185d46 + da48ff2 commit fb81408
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
GOFLAGS: -tags=functional
uses: golangci/golangci-lint-action@v3
with:
version: v1.48.0
version: v1.53.3
test:
name: Unit Testing with Go ${{ matrix.go-version }}
runs-on: ubuntu-latest
Expand Down
15 changes: 11 additions & 4 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ linters-settings:
misspell:
locale: US
goimports:
local-prefixes: github.com/Shopify/sarama
local-prefixes: github.com/IBM/sarama
gocritic:
enabled-tags:
- diagnostic
Expand All @@ -39,11 +39,18 @@ linters-settings:
lines: 300
statements: 300

depguard:
rules:
main:
deny:
- pkg: "io/ioutil"
desc: Use the "io" and "os" packages instead.

linters:
disable-all: true
enable:
- bodyclose
- deadcode
# - deadcode
- depguard
- exportloopref
- dogsled
Expand All @@ -68,12 +75,12 @@ linters:
# - paralleltest
# - scopelint
- staticcheck
- structcheck
# - structcheck
# - stylecheck
- typecheck
- unconvert
- unused
- varcheck
# - varcheck
- whitespace

issues:
Expand Down

0 comments on commit fb81408

Please sign in to comment.