golangci-lint

Add delegation plan test #2121

Triggered via pull request October 18, 2023 09:23

joe-bowman

synchronize #709

delegation-plan-changes

Status Failure

Total duration 1m 37s

Artifacts –

This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

golangci.yml

on: pull_request

6 errors and 2 warnings

lint: x/interchainstaking/keeper/delegation_test.go#L660

Error return value of `quicksilver.InterchainstakingKeeper.SetValidator` is not checked (errcheck)

lint: x/interchainstaking/keeper/delegation_test.go#L689

File is not `gofumpt`-ed (gofumpt)

lint: x/interchainstaking/keeper/delegation_test.go#L696

File is not `gofumpt`-ed (gofumpt)

lint: x/interchainstaking/keeper/delegation_test.go#L15

ST1019: package "github.com/quicksilver-zone/quicksilver/x/interchainstaking/types" is being imported more than once (stylecheck)

lint: x/interchainstaking/keeper/keeper_test.go#L62

ST1016: methods on the same type should have the same receiver name (seen 1x "s", 87x "suite") (stylecheck)

issues found

lint: x/interchainstaking/keeper/delegation_test.go#L16

duplicated-imports: Package "github.com/quicksilver-zone/quicksilver/x/interchainstaking/types" already imported (revive)

lint: x/interchainstaking/keeper/delegation_test.go#L634

receiver-naming: receiver name s should be consistent with previous receiver name suite for KeeperTestSuite (revive)