Preparing report...

Report for github.com/UptickNetwork/uptick

(v0.0.0-20220317024937-66f7190b520d)

A+    Excellent!    Found 40 issues across 134 files

Tweet

gofmt100%

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

No problems detected. Good job!


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!


gocyclo94%

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.

    • app/export.go
    • Line 66: warning: cyclomatic complexity 18 of function (*Uptick).prepForZeroHeightGenesis() is high (> 15) (gocyclo)

golint73%

Golint is a linter for Go source code.

    • version/version.go
    • Line 9: warning: exported var AppVersion should have comment or be unexported (golint)
    • Line 26: warning: exported function Version should have comment or be unexported (golint)
    • testutil/network/network.go
    • Line 198: warning: exported type CLILogger should have comment or be unexported (golint)
    • Line 202: warning: exported method CLILogger.Log should have comment or be unexported (golint)
    • Line 206: warning: exported method CLILogger.Logf should have comment or be unexported (golint)
    • Line 210: warning: exported function NewCLILogger should have comment or be unexported (golint)
    • x/collection/types/owners.go
    • Line 28: warning: comment on exported type IDCollections should be of the form "IDCollections ..." (with optional leading article) (golint)
    • Line 70: warning: exported type Owners should have comment or be unexported (golint)
    • Line 72: warning: comment on exported function NewOwners should be of the form "NewOwners ..." (golint)
    • x/collection/keeper/grpc_query.go
    • Line 18: warning: exported method Keeper.Supply should have comment or be unexported (golint)
    • Line 35: warning: exported method Keeper.NFTsOfOwner should have comment or be unexported (golint)
    • Line 73: warning: exported method Keeper.Collection should have comment or be unexported (golint)
    • Line 121: warning: exported method Keeper.Denom should have comment or be unexported (golint)
    • Line 130: warning: exported method Keeper.Denoms should have comment or be unexported (golint)
    • Line 155: warning: exported method Keeper.NFT should have comment or be unexported (golint)
    • x/erc20/module.go
    • Line 35: warning: comment on exported type AppModuleBasic should be of the form "AppModuleBasic ..." (with optional leading article) (golint)
    • Line 38: warning: exported method AppModuleBasic.Name should have comment or be unexported (golint)
    • Line 61: warning: exported method AppModuleBasic.ValidateGenesis should have comment or be unexported (golint)
    • Line 74: warning: exported method AppModuleBasic.RegisterGRPCGatewayRoutes should have comment or be unexported (golint)
    • Line 90: warning: exported type AppModule should have comment or be unexported (golint)
    • Line 108: warning: exported method AppModule.Name should have comment or be unexported (golint)
    • Line 112: warning: exported method AppModule.RegisterInvariants should have comment or be unexported (golint)
    • Line 114: warning: exported method AppModule.NewHandler should have comment or be unexported (golint)
    • Line 118: warning: exported method AppModule.Route should have comment or be unexported (golint)
    • Line 122: warning: exported method AppModule.QuerierRoute should have comment or be unexported (golint)
    • Line 126: warning: exported method AppModule.LegacyQuerierHandler should have comment or be unexported (golint)
    • Line 130: warning: exported method AppModule.RegisterServices should have comment or be unexported (golint)
    • Line 137: warning: exported method AppModule.BeginBlock should have comment or be unexported (golint)
    • Line 139: warning: exported method AppModule.EndBlock should have comment or be unexported (golint)
    • Line 143: warning: exported method AppModule.InitGenesis should have comment or be unexported (golint)
    • Line 150: warning: exported method AppModule.ExportGenesis should have comment or be unexported (golint)
    • Line 155: warning: exported method AppModule.GenerateGenesisState should have comment or be unexported (golint)
    • Line 157: warning: exported method AppModule.ProposalContents should have comment or be unexported (golint)
    • Line 161: warning: exported method AppModule.RandomizedParams should have comment or be unexported (golint)
    • Line 165: warning: exported method AppModule.RegisterStoreDecoder should have comment or be unexported (golint)
    • Line 167: warning: exported method AppModule.WeightedOperations should have comment or be unexported (golint)
    • x/nft/codec.go
    • Line 9: warning: exported function RegisterInterfaces should have comment or be unexported (golint)
    • x/nft/module/module.go
    • Line 127: warning: exported method AppModule.NewHandler should have comment or be unexported (golint)
    • Line 160: warning: exported method AppModule.BeginBlock should have comment or be unexported (golint)
    • x/erc20/types/msg.go
    • Line 18: warning: exported const TypeMsgConvertCoin should have comment (or a comment on this block) or be unexported (golint)
    • x/collection/client/testutil/test_helpers.go
    • Line 27: warning: exported function BurnNFTExec should have comment or be unexported (golint)
    • Line 38: warning: exported function MintNFTExec should have comment or be unexported (golint)
    • Line 49: warning: exported function EditNFTExec should have comment or be unexported (golint)
    • Line 60: warning: exported function TransferNFTExec should have comment or be unexported (golint)
    • Line 72: warning: exported function QueryDenomExec should have comment or be unexported (golint)
    • Line 82: warning: exported function QueryCollectionExec should have comment or be unexported (golint)
    • Line 92: warning: exported function QueryDenomsExec should have comment or be unexported (golint)
    • Line 101: warning: exported function QuerySupplyExec should have comment or be unexported (golint)
    • Line 111: warning: exported function QueryOwnerExec should have comment or be unexported (golint)
    • Line 121: warning: exported function QueryNFTExec should have comment or be unexported (golint)
    • Line 132: warning: exported function TransferDenomExec should have comment or be unexported (golint)
    • app/app.go
    • Line 704: warning: comment on exported method Uptick.DeliverTx should be of the form "DeliverTx ..." (golint)
    • Line 833: warning: exported method Uptick.RegisterTxService should have comment or be unexported (golint)
    • Line 837: warning: exported method Uptick.RegisterTendermintService should have comment or be unexported (golint)
    • x/nft/keys.go
    • Line 4: warning: comment on exported const ModuleName should be of the form "ModuleName ..." (golint)
    • x/nft/msgs.go
    • Line 12: warning: comment on exported method MsgSend.ValidateBasic should be of the form "ValidateBasic ..." (golint)
    • x/erc20/client/rest/rest.go
    • Line 54: warning: exported function RegisterCoinProposalRESTHandler should have comment or be unexported (golint)
    • Line 61: warning: exported function RegisterERC20ProposalRESTHandler should have comment or be unexported (golint)
    • Line 68: warning: exported function ToggleTokenRelayRESTHandler should have comment or be unexported (golint)
    • Line 75: warning: exported function UpdateTokenPairERC20ProposalRESTHandler should have comment or be unexported (golint)
    • x/collection/types/codec.go
    • Line 12: warning: exported var ModuleCdc should have comment or be unexported (golint)
    • Line 14: warning: exported function RegisterInterfaces should have comment or be unexported (golint)
    • x/collection/types/collection.go
    • Line 22: warning: exported method Collection.Supply should have comment or be unexported (golint)
    • Line 26: warning: comment on exported function NewCollections should be of the form "NewCollections ..." (golint)
    • x/erc20/client/cli/query.go
    • Line 69: warning: comment on exported function GetTokenPairCmd should be of the form "GetTokenPairCmd ..." (golint)
    • Line 101: warning: comment on exported function GetParamsCmd should be of the form "GetParamsCmd ..." (golint)
    • x/erc20/types/events.go
    • Line 29: warning: comment on exported type LogTransfer should be of the form "LogTransfer ..." (with optional leading article) (golint)
    • x/collection/client/cli/flags.go
    • Line 8: warning: exported const FlagTokenName should have comment (or a comment on this block) or be unexported (golint)
    • Line 23: warning: exported var FsIssueDenom should have comment or be unexported (golint)
    • x/erc20/types/params.go
    • Line 31: warning: exported function DefaultParams should have comment or be unexported (golint)
    • Line 55: warning: exported method Params.Validate should have comment or be unexported (golint)
    • x/collection/types/querier.go
    • Line 44: warning: comment on exported function NewQueryOwnerParams should be of the form "NewQueryOwnerParams ..." (golint)
    • Line 52: warning: comment on exported type QueryCollectionParams should be of the form "QueryCollectionParams ..." (with optional leading article) (golint)
    • cmd/uptickd/root.go
    • Line 46: warning: exported const EnvPrefix should have comment (or a comment on this block) or be unexported (golint)
    • contracts/erc20.go
    • Line 14: warning: comment on exported var ERC20MinterBurnerDecimalsJSON should be of the form "ERC20MinterBurnerDecimalsJSON ..." (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!