gofmt97%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command

• ibc-go/modules/light-clients/09-localhost/types/localhost_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/testing/simapp/simd/cmd/cmd_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/core/03-connection/simulation/decoder_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/core/04-channel/simulation/decoder_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/core/05-port/keeper/keeper_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/core/simulation/decoder_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/apps/transfer/keeper/keeper.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/core/02-client/simulation/decoder_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• ibc-go/modules/apps/transfer/simulation/decoder_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo96%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

• ibc-go/modules/core/03-connection/types/msgs.go
  • Line 102: warning: cyclomatic complexity 17 of function (MsgConnectionOpenTry).ValidateBasic() is high (> 15) (gocyclo)

• ibc-go/modules/core/04-channel/types/genesis.go
  • Line 76: warning: cyclomatic complexity 21 of function (GenesisState).Validate() is high (> 15) (gocyclo)

• ibc-go/testing/simapp/simd/cmd/genaccounts.go
  • Line 31: warning: cyclomatic complexity 22 of function AddGenesisAccountCmd() is high (> 15) (gocyclo)

• ibc-go/testing/simapp/simd/cmd/testnet.go
  • Line 100: warning: cyclomatic complexity 18 of function InitTestnet() is high (> 15) (gocyclo)

• ibc-go/modules/light-clients/06-solomachine/types/codec.go
  • Line 46: warning: cyclomatic complexity 22 of function UnmarshalDataByType() is high (> 15) (gocyclo)

• ibc-go/modules/light-clients/07-tendermint/types/misbehaviour.go
  • Line 51: warning: cyclomatic complexity 16 of function (Misbehaviour).ValidateBasic() is high (> 15) (gocyclo)

• ibc-go/modules/core/02-client/types/genesis.go
  • Line 109: warning: cyclomatic complexity 26 of function (GenesisState).Validate() is high (> 15) (gocyclo)

• ibc-go/modules/core/03-connection/keeper/handshake.go
  • Line 62: warning: cyclomatic complexity 18 of function (Keeper).ConnOpenTry() is high (> 15) (gocyclo)

• ibc-go/testing/simapp/state.go
  • Line 28: warning: cyclomatic complexity 18 of function AppStateFn() is high (> 15) (gocyclo)

• ibc-go/testing/simapp/export.go
  • Line 50: warning: cyclomatic complexity 17 of function (*SimApp).prepForZeroHeightGenesis() is high (> 15) (gocyclo)

golint93%

Golint is a linter for Go source code.

• ibc-go/modules/light-clients/07-tendermint/types/errors.go
  • Line 8: warning: exported const SubModuleName should have comment (or a comment on this block) or be unexported (golint)

• ibc-go/testing/coordinator.go
  • Line 13: warning: exported const ChainIDPrefix should have comment or be unexported (golint)
  • Line 17: warning: exported var TimeIncrement should have comment or be unexported (golint)
  • Line 95: warning: comment on exported method Coordinator.SetupConnections should be of the form "SetupConnections ..." (golint)
  • Line 104: warning: comment on exported method Coordinator.CreateConnections should be of the form "CreateConnections ..." (golint)
  • Line 147: warning: comment on exported method Coordinator.CreateChannels should be of the form "CreateChannels ..." (golint)

• ibc-go/testing/simapp/test_helpers.go
  • Line 117: warning: exported type GenerateAccountStrategy should have comment or be unexported (golint)
  • Line 167: warning: comment on exported function AddTestAddrsIncremental should be of the form "AddTestAddrsIncremental ..." (golint)
  • Line 208: warning: exported function TestAddr should have comment or be unexported (golint)

• ibc-go/modules/core/05-port/types/router.go
  • Line 9: warning: comment on exported type Router should be of the form "Router ..." (with optional leading article) (golint)
  • Line 16: warning: exported function NewRouter should have comment or be unexported (golint)

• ibc-go/modules/core/02-client/client/proposal_handler.go
  • Line 14: warning: exported var UpdateClientProposalHandler should have comment or be unexported (golint)

• ibc-go/testing/mock/mock.go
  • Line 25: warning: exported const ModuleName should have comment (or a comment on this block) or be unexported (golint)
  • Line 29: warning: exported var MockAcknowledgement should have comment or be unexported (golint)
  • Line 37: warning: comment on exported type PortKeeper should be of the form "PortKeeper ..." (with optional leading article) (golint)

• ibc-go/modules/light-clients/07-tendermint/types/misbehaviour.go
  • Line 17: warning: comment on exported var FrozenHeight should be of the form "FrozenHeight ..." (golint)

• ibc-go/testing/app.go
  • Line 30: warning: exported var DefaultTestingAppInit should have comment or be unexported (golint)
  • Line 32: warning: exported type TestingApp should have comment or be unexported (golint)
  • Line 50: warning: exported function SetupTestingApp should have comment or be unexported (golint)

• ibc-go/testing/chain.go
  • Line 143: warning: comment on exported method TestChain.QueryProofAtHeight should be of the form "QueryProofAtHeight ..." (golint)
  • Line 319: warning: comment on exported method TestChain.ConstructUpdateTMClientHeaderWithTrustedHeight should be of the form "ConstructUpdateTMClientHeaderWithTrustedHeight ..." (golint)

• ibc-go/testing/config.go
  • Line 13: warning: exported type ClientConfig should have comment or be unexported (golint)
  • Line 17: warning: exported type TendermintConfig should have comment or be unexported (golint)
  • Line 26: warning: exported function NewTendermintConfig should have comment or be unexported (golint)
  • Line 37: warning: exported method TendermintConfig.GetClientType should have comment or be unexported (golint)
  • Line 41: warning: exported type ConnectionConfig should have comment or be unexported (golint)
  • Line 46: warning: exported function NewConnectionConfig should have comment or be unexported (golint)
  • Line 53: warning: exported type ChannelConfig should have comment or be unexported (golint)
  • Line 59: warning: exported function NewChannelConfig should have comment or be unexported (golint)

• ibc-go/modules/core/02-client/types/proposal.go
  • Line 16: warning: exported const ProposalTypeUpgrade should have comment (or a comment on this block) or be unexported (golint)

• ibc-go/modules/light-clients/06-solomachine/types/codec.go
  • Line 33: warning: exported function UnmarshalSignatureData should have comment or be unexported (golint)

• ibc-go/testing/values.go
  • Line 1: warning: package comment should be of the form "Package ibctesting ..." (golint)
  • Line 1: warning: package comment should not have leading space (golint)
  • Line 20: warning: exported const FirstClientID should have comment (or a comment on this block) or be unexported (golint)
  • Line 24: warning: comment on exported const TrustingPeriod should be of the form "TrustingPeriod ..." (golint)
  • Line 33: warning: comment on exported const TransferPort should be of the form "TransferPort ..." (golint)
  • Line 37: warning: comment on exported const Title should be of the form "Title ..." (golint)
  • Line 43: warning: exported var DefaultOpenInitVersion should have comment or be unexported (golint)
  • Line 45: warning: comment on exported var DefaultTrustLevel should be of the form "DefaultTrustLevel ..." (golint)

• ibc-go/testing/simapp/simd/cmd/testnet.go
  • Line 99: warning: comment on exported function InitTestnet should be of the form "InitTestnet ..." (golint)

• ibc-go/testing/mock/doc.go
  • Line 1: warning: package comment should be of the form "Package mock ..." (golint)

• ibc-go/testing/mock/privval.go
  • Line 15: warning: comment on exported type PV should be of the form "PV ..." (with optional leading article) (golint)
  • Line 21: warning: exported function NewPV should have comment or be unexported (golint)

• ibc-go/modules/core/04-channel/keeper/handshake.go
  • Line 381: warning: comment on exported method Keeper.ChanCloseInit should be of the form "ChanCloseInit ..." (golint)

• ibc-go/modules/light-clients/06-solomachine/types/errors.go
  • Line 8: warning: exported const SubModuleName should have comment (or a comment on this block) or be unexported (golint)
  • Line 12: warning: exported var ErrInvalidHeader should have comment or be unexported (golint)

• ibc-go/modules/light-clients/07-tendermint/types/store.go
  • Line 34: warning: exported const KeyIterateConsensusStatePrefix should have comment or be unexported (golint)

• ibc-go/testing/simapp/genesis.go
  • Line 9: warning: comment on exported type GenesisState should be of the form "GenesisState ..." (with optional leading article) (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign98%

IneffAssign detects ineffectual assignments in Go code.

• ibc-go/modules/core/02-client/keeper/client_test.go
  • Line 674: warning: ineffectual assignment to err (ineffassign)

• ibc-go/modules/core/02-client/keeper/params_test.go
  • Line 15: warning: ineffectual assignment to params (ineffassign)

• ibc-go/modules/core/03-connection/keeper/params_test.go
  • Line 15: warning: ineffectual assignment to params (ineffassign)

• ibc-go/modules/core/04-channel/keeper/handshake_test.go
  • Line 657: warning: ineffectual assignment to err (ineffassign)

misspell99%

Misspell Finds commonly misspelled English words

• ibc-go/modules/apps/transfer/module.go
  • Line 321: warning: "succesfully" is a misspelling of "successfully" (misspell)

• ibc-go/modules/light-clients/06-solomachine/types/client_state_test.go
  • Line 30: warning: "arguements" is a misspelling of "arguments" (misspell)