Skip to content

Commit

Permalink
Fix failing pre-commit checks (#45)
Browse files Browse the repository at this point in the history
  • Loading branch information
martin-majlis-s1 authored Jul 27, 2023
1 parent 86db63b commit aca8af3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
22 changes: 2 additions & 20 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,33 +22,15 @@ repos:
test/testdata/buffer_test_payload_full.json
)
- id: trailing-whitespace
- repo: https://github.com/dnephin/pre-commit-golang
rev: v0.5.1
hooks:
- id: go-fmt
- id: go-imports
- id: go-cyclo
- id: validate-toml
- id: no-go-testing
- id: golangci-lint
# - id: go-critic
# # args: [-disable=sloppyTypeAssert]
# args: ["-v", "-enable='yodaStyleExpr'", "-disable=sloppyTypeAssert"]
# there is timeout of 30s for go-unit-test and we need more
# - id: go-unit-tests
- id: go-build
- id: go-mod-tidy
- id: go-mod-vendor

- repo: https://github.com/Bahjat/pre-commit-golang
rev: v1.0.3
hooks:
- id: go-fmt-import
- id: go-lint
# - id: go-unit-tests
- id: gofumpt
- id: go-err-check
# - id: go-static-check
# - id: golangci-lint
- id: golangci-lint
- id: go-static-check
# - id: go-ruleguard # requires https://github.com/quasilyte/go-ruleguard
# args: [rules/rules.go] # required
2 changes: 1 addition & 1 deletion scripts/install-dev-tools.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ go install -v github.com/BurntSushi/toml/cmd/tomlv@latest
go install -v mvdan.cc/gofumpt@latest
go install -v github.com/kisielk/errcheck@latest
# go install -v github.com/quasilyte/go-ruleguard/cmd/ruleguard@latest
# go install -v honnef.co/go/tools/cmd/staticcheck@latest
go install -v honnef.co/go/tools/cmd/staticcheck@latest

0 comments on commit aca8af3

Please sign in to comment.