test: Validate EIP712 patch in tests (#2044) #2905
ci-commit.yml
on: push
lint
/
...
/
Check Proto
3m 25s
lint
/
golangci-lint
3m 57s
Annotations
10 errors and 2 warnings
|
lint / golangci-lint:
cmd/kava/cmd/iavlviewer/shape.go#L45
Error return value of `tree.RenderShape` is not checked (errcheck)
|
|
lint / golangci-lint:
cmd/kava/cmd/shard.go#L223
Error return value of `multistore.PruneStores` is not checked (errcheck)
|
|
lint / golangci-lint:
x/pricefeed/keeper/keeper.go#L185
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint / golangci-lint:
x/evmutil/keeper/conversion_evm_native_bep3_test.go#L178
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint / golangci-lint:
x/evmutil/keeper/msg_server_bep3_test.go#L103
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
lint / golangci-lint:
x/evmutil/keeper/msg_server_bep3_test.go#L109
G601: Implicit memory aliasing in for loop. (gosec)
|
|
lint / golangci-lint:
x/evmutil/keeper/msg_server_bep3_test.go#L272
G601: Implicit memory aliasing in for loop. (gosec)
|
|
lint / golangci-lint:
tests/util/evmsigner.go#L84
var-naming: func parameter evmChainId should be evmChainID (revive)
|
|
lint / golangci-lint:
x/cdp/keeper/interest.go#L267
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint / golangci-lint:
app/upgrades.go#L51
unused-parameter: parameter 'plan' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint / proto-lint / Check Proto
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint / golangci-lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|