Skip to content

gci linter

gci linter #724

Triggered via pull request July 24, 2023 14:25
@faddatfaddat
synchronize #179
faddat/gci
Status Failure
Total duration 4m 24s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
lint: app/app.go#L692
singleCaseSwitch: found switch with default case only (gocritic)
lint: app/ibctesting/event_utils.go#L81
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L84
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L87
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L90
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L93
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L101
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L68
unnecessary conversion (unconvert)
lint: app/ibctesting/event_utils.go#L73
unnecessary conversion (unconvert)
lint: x/ratelimit/keeper/abci.go#L20
naked return in func `BeginBlocker.<func():15>` with 41 lines of code (nakedret)
lint: x/ibc-hooks/client/cli/query.go#L15
unused-parameter: parameter 'args' seems to be unused, consider removing or renaming it as _ (revive)
lint: x/ratelimit/types/epoch.go#L9
var-naming: don't use ALL_CAPS in Go names; use CamelCase (revive)
lint: x/ratelimit/types/keys.go#L33
var-naming: func parameter channelId should be channelID (revive)
lint: x/ratelimit/types/keys.go#L34
var-naming: var channelIdBz should be channelIDBz (revive)
lint: x/ratelimit/types/epoch.go#L10
var-declaration: should omit type time.Duration from declaration of var EpochHourPeriod; it will be inferred from the right-hand side (revive)
lint: x/ratelimit/types/msg.go#L193
if-return: redundant if ...; err != nil check, just return error instead. (revive)
lint: x/ratelimit/types/msg.go#L239
if-return: redundant if ...; err != nil check, just return error instead. (revive)
lint: x/ratelimit/types/keys.go#L30
var-declaration: should omit type int from declaration of var PendingSendPacketChannelLength; it will be inferred from the right-hand side (revive)
lint: x/ratelimit/keeper/grpc_query.go#L16
unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive)
lint: x/ratelimit/keeper/keeper.go#L58
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)