Merge pull request #104 from decentrio/intergrate-multistaking #144
jiujiteiroAnnotations
10 errors and 5 warnings
|
lint:
app/upgrades/multi-staking/upgrade.go#L97
Error return value of `bk.SendCoins` is not checked (errcheck)
|
|
lint:
app/upgrades/multi-staking/upgrade.go#L104
Error return value of `bk.MintCoins` is not checked (errcheck)
|
|
lint:
app/upgrades/multi-staking/upgrade.go#L105
Error return value of `bk.SendCoins` is not checked (errcheck)
|
|
lint:
app/upgrades.go#L40
singleCaseSwitch: should rewrite switch statement to if statement (gocritic)
|
|
lint:
app/upgrades/multi-staking/upgrade.go#L167
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
x/mint/keeper/keeper_test.go#L19
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
x/mint/keeper/keeper_test.go#L24
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
x/mint/keeper/keeper_test.go#L105
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
x/mint/keeper/keeper_test.go#L120
File is not `gofumpt`-ed (gofumpt)
|
|
lint:
x/mint/keeper/keeper_test.go#L135
File is not `gofumpt`-ed (gofumpt)
|
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4, actions/checkout@v3, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
lint:
app/upgrades/multi-staking/types.go#L74
exported: type name will be used as multistaking.MultiStakingGenesisState by other packages, and that stutters; consider calling this GenesisState (revive)
|
|
lint:
app/upgrades/multi-staking/upgrade.go#L46
unused-parameter: parameter 'msk' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
app/upgrades/multi-staking/upgrade.go#L48
unused-parameter: parameter 'keys' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
app/test_helpers.go#L173
var-naming: var lockId should be lockID (revive)
|