Preparing report...

Report for github.com/makerdao/vulcanizedb

A+    Excellent!    Found 113 issues across 193 files

Tweet

gofmt98%

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


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!


gocyclo99%

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.


golint41%

Golint is a linter for Go source code.

    • vulcanizedb/pkg/history/validation_window.go
    • Line 25: warning: exported type ValidationWindow should have comment or be unexported (golint)
    • Line 30: warning: exported method ValidationWindow.Size should have comment or be unexported (golint)
    • Line 34: warning: exported function MakeValidationWindow should have comment or be unexported (golint)
    • Line 44: warning: exported function MakeRange should have comment or be unexported (golint)
    • Line 52: warning: exported method ValidationWindow.GetString should have comment or be unexported (golint)
    • vulcanizedb/pkg/datastore/postgres/repositories/checked_logs_repository.go
    • Line 24: warning: exported type CheckedLogsRepository should have comment or be unexported (golint)
    • Line 28: warning: exported function NewCheckedLogsRepository should have comment or be unexported (golint)
    • Line 32: warning: comment on exported method CheckedLogsRepository.AlreadyWatchingLog should be of the form "AlreadyWatchingLog ..." (golint)
    • Line 52: warning: comment on exported method CheckedLogsRepository.MarkLogWatched should be of the form "MarkLogWatched ..." (golint)
    • vulcanizedb/pkg/datastore/postgres/repositories/event_log_repository.go
    • Line 36: warning: exported type EventLogRepository should have comment or be unexported (golint)
    • Line 40: warning: exported function NewEventLogRepository should have comment or be unexported (golint)
    • Line 61: warning: exported method EventLogRepository.GetUntransformedEventLogs should have comment or be unexported (golint)
    • Line 104: warning: exported method EventLogRepository.CreateEventLogs should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/test_data/test_helpers.go
    • Line 1: warning: don't use an underscore in package name (golint)
    • Line 15: warning: should not use dot imports (golint)
    • Line 18: warning: comment on exported function CreateTestLog should be of the form "CreateTestLog ..." (golint)
    • Line 72: warning: exported function CreateMatchingTx should have comment or be unexported (golint)
    • Line 93: warning: comment on exported function ExpectCheckedHeadersInThisSchema should be of the form "ExpectCheckedHeadersInThisSchema ..." (golint)
    • vulcanizedb/pkg/core/node_info.go
    • Line 23: warning: exported type NodeType should have comment or be unexported (golint)
    • Line 26: warning: exported const GETH should have comment (or a comment on this block) or be unexported (golint)
    • Line 33: warning: exported const KOVAN_NETWORK_ID should have comment (or a comment on this block) or be unexported (golint)
    • Line 36: warning: exported type Node should have comment or be unexported (golint)
    • Line 43: warning: exported type ParityNodeInfo should have comment or be unexported (golint)
    • Line 53: warning: exported type ParityVersion should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/streamer.go
    • Line 9: warning: exported type MockStoragediffStreamer should have comment or be unexported (golint)
    • Line 16: warning: exported method MockStoragediffStreamer.Stream should have comment or be unexported (golint)
    • Line 28: warning: exported method MockStoragediffStreamer.SetSubscribeError should have comment or be unexported (golint)
    • Line 32: warning: exported method MockStoragediffStreamer.SetPayloads should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/factories/event/transformer.go
    • Line 26: warning: exported type ITransformer should have comment or be unexported (golint)
    • Line 31: warning: exported type TransformerInitializer should have comment or be unexported (golint)
    • Line 33: warning: exported type TransformerConfig should have comment or be unexported (golint)
    • Line 42: warning: exported function HexStringsToAddresses should have comment or be unexported (golint)
    • vulcanizedb/pkg/plugin/manager/manager.go
    • Line 33: warning: comment on exported type MigrationManager should be of the form "MigrationManager ..." (with optional leading article) (golint)
    • Line 45: warning: comment on exported function NewMigrationManager should be of the form "NewMigrationManager ..." (golint)
    • vulcanizedb/pkg/config/plugin.go
    • Line 31: warning: exported type Plugin should have comment or be unexported (golint)
    • Line 40: warning: exported type Transformer should have comment or be unexported (golint)
    • Line 49: warning: exported var PluginFilePath should have comment or be unexported (golint)
    • Line 50: warning: error var MissingPathErr should have name of the form ErrFoo (golint)
    • Line 51: warning: error var MissingRepositoryErr should have name of the form ErrFoo (golint)
    • Line 52: warning: error var MissingMigrationsErr should have name of the form ErrFoo (golint)
    • Line 53: warning: error var MissingRankErr should have name of the form ErrFoo (golint)
    • Line 54: warning: error var RankParsingErr should have name of the form ErrFoo (golint)
    • Line 55: warning: error var MissingTypeErr should have name of the form ErrFoo (golint)
    • Line 56: warning: error var UnknownTransformerTypeErr should have name of the form ErrFoo (golint)
    • Line 59: warning: exported function PreparePluginConfig should have comment or be unexported (golint)
    • Line 114: warning: exported method Plugin.GetPluginPaths should have comment or be unexported (golint)
    • Line 127: warning: comment on exported method Plugin.GetMigrationsPaths should be of the form "GetMigrationsPaths ..." (golint)
    • Line 144: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 168: warning: comment on exported method Plugin.GetRepoPaths should be of the form "GetRepoPaths ..." (golint)
    • Line 178: warning: exported type TransformerType should have comment or be unexported (golint)
    • Line 181: warning: exported const UnknownTransformerType should have comment (or a comment on this block) or be unexported (golint)
    • Line 202: warning: exported function GetTransformerType should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_rpc_client.go
    • Line 31: warning: should not use dot imports (golint)
    • Line 34: warning: exported type MockRpcClient should have comment or be unexported (golint)
    • Line 57: warning: exported function NewMockRpcClient should have comment or be unexported (golint)
    • Line 61: warning: exported method MockRpcClient.Subscribe should have comment or be unexported (golint)
    • Line 78: warning: exported method MockRpcClient.AssertSubscribeCalledWith should have comment or be unexported (golint)
    • Line 84: warning: exported method MockRpcClient.SetIpcPath should have comment or be unexported (golint)
    • Line 88: warning: exported method MockRpcClient.BatchCall should have comment or be unexported (golint)
    • Line 111: warning: exported method MockRpcClient.CallContext should have comment or be unexported (golint)
    • Line 147: warning: exported method MockRpcClient.IpcPath should have comment or be unexported (golint)
    • Line 151: warning: exported method MockRpcClient.SetCallContextErr should have comment or be unexported (golint)
    • Line 155: warning: exported method MockRpcClient.SetReturnPOAHeader should have comment or be unexported (golint)
    • Line 159: warning: exported method MockRpcClient.SetReturnPOWHeaders should have comment or be unexported (golint)
    • Line 163: warning: exported method MockRpcClient.SetReturnPOAHeaders should have comment or be unexported (golint)
    • Line 167: warning: exported method MockRpcClient.AssertCallContextCalledWith should have comment or be unexported (golint)
    • Line 173: warning: exported method MockRpcClient.AssertBatchCalledWith should have comment or be unexported (golint)
    • vulcanizedb/pkg/datastore/repository.go
    • Line 25: warning: exported type AddressRepository should have comment or be unexported (golint)
    • Line 29: warning: exported type CheckedHeadersRepository should have comment or be unexported (golint)
    • Line 35: warning: exported type CheckedLogsRepository should have comment or be unexported (golint)
    • Line 40: warning: exported type HeaderRepository should have comment or be unexported (golint)
    • Line 52: warning: exported type EventLogRepository should have comment or be unexported (golint)
    • vulcanizedb/test_config/test_config.go
    • Line 17: warning: don't use an underscore in package name (golint)
    • Line 31: warning: exported var TestConfig should have comment or be unexported (golint)
    • Line 32: warning: exported var DBConfig should have comment or be unexported (golint)
    • Line 33: warning: exported var ABIFilePath should have comment or be unexported (golint)
    • Line 65: warning: exported function NewTestDB should have comment or be unexported (golint)
    • Line 73: warning: exported function CleanTestDB should have comment or be unexported (golint)
    • Line 86: warning: comment on exported function NewTestNode should be of the form "NewTestNode ..." (golint)
    • vulcanizedb/cmd/root.go
    • Line 43: warning: exported var LogWithCommand should have comment or be unexported (golint)
    • Line 67: warning: exported function Execute should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/transactions/syncer.go
    • Line 28: warning: exported type ITransactionsSyncer should have comment or be unexported (golint)
    • Line 32: warning: exported type TransactionsSyncer should have comment or be unexported (golint)
    • Line 37: warning: exported function NewTransactionsSyncer should have comment or be unexported (golint)
    • Line 45: warning: exported method TransactionsSyncer.SyncTransactions should have comment or be unexported (golint)
    • vulcanizedb/pkg/plugin/builder/builder.go
    • Line 30: warning: comment on exported type PluginBuilder should be of the form "PluginBuilder ..." (with optional leading article) (golint)
    • Line 45: warning: comment on exported function NewPluginBuilder should be of the form "NewPluginBuilder ..." (golint)
    • Line 71: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 97: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • vulcanizedb/pkg/eth/client/rpc_client.go
    • Line 28: warning: exported type RpcClient should have comment or be unexported (golint)
    • Line 33: warning: exported function NewRpcClient should have comment or be unexported (golint)
    • Line 40: warning: exported method RpcClient.CallContext should have comment or be unexported (golint)
    • Line 46: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 51: warning: exported method RpcClient.IpcPath should have comment or be unexported (golint)
    • Line 55: warning: exported method RpcClient.BatchCall should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_fs_reader.go
    • Line 19: warning: should not use dot imports (golint)
    • Line 21: warning: exported type MockFsReader should have comment or be unexported (golint)
    • Line 28: warning: exported method MockFsReader.SetReturnBytes should have comment or be unexported (golint)
    • Line 32: warning: exported method MockFsReader.SetReturnErr should have comment or be unexported (golint)
    • Line 42: warning: exported method MockFsReader.AssertReadCalledWith should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/chunker/log_chunker.go
    • Line 27: warning: exported type Chunker should have comment or be unexported (golint)
    • Line 32: warning: exported type LogChunker should have comment or be unexported (golint)
    • Line 37: warning: comment on exported function NewLogChunker should be of the form "NewLogChunker ..." (golint)
    • Line 46: warning: comment on exported method LogChunker.AddConfig should be of the form "AddConfig ..." (golint)
    • Line 55: warning: comment on exported method LogChunker.ChunkLogs should be of the form "ChunkLogs ..." (golint)
    • vulcanizedb/pkg/fakes/mock_checked_headers_repository.go
    • Line 23: warning: exported type MockCheckedHeadersRepository should have comment or be unexported (golint)
    • Line 33: warning: exported method MockCheckedHeadersRepository.MarkSingleHeaderUnchecked should have comment or be unexported (golint)
    • Line 37: warning: exported method MockCheckedHeadersRepository.MarkHeaderChecked should have comment or be unexported (golint)
    • Line 42: warning: exported method MockCheckedHeadersRepository.UncheckedHeaders should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/storage/backfill/storage_value_loader.go
    • Line 21: warning: exported var MaxRequestSize should have comment or be unexported (golint)
    • Line 26: warning: exported function NewStorageValueLoader should have comment or be unexported (golint)
    • Line 43: warning: exported type StorageValueLoader should have comment or be unexported (golint)
    • Line 54: warning: exported method StorageValueLoader.Run should have comment or be unexported (golint)
    • Line 119: warning: should omit 2nd value from range; this loop is equivalent to `for key := range ...` (golint)
    • vulcanizedb/pkg/datastore/postgres/repositories/checked_headers_repository.go
    • Line 36: warning: exported type SchemaQuery should have comment or be unexported (golint)
    • Line 40: warning: exported type CheckedHeadersRepository should have comment or be unexported (golint)
    • Line 45: warning: exported function NewCheckedHeadersRepository should have comment or be unexported (golint)
    • Line 57: warning: comment on exported method CheckedHeadersRepository.MarkHeaderChecked should be of the form "MarkHeaderChecked ..." (golint)
    • Line 64: warning: comment on exported method CheckedHeadersRepository.MarkSingleHeaderUnchecked should be of the form "MarkSingleHeaderUnchecked ..." (golint)
    • Line 76: warning: comment on exported method CheckedHeadersRepository.UncheckedHeaders should be of the form "UncheckedHeaders ..." (golint)
    • vulcanizedb/pkg/plugin/helpers/helpers.go
    • Line 30: warning: exported function CleanPath should have comment or be unexported (golint)
    • Line 44: warning: exported function ClearFiles should have comment or be unexported (golint)
    • Line 61: warning: exported function CopyFile should have comment or be unexported (golint)
    • Line 78: warning: exported function CopyDir should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_blockchain.go
    • Line 26: warning: should not use dot imports (golint)
    • Line 29: warning: exported type MockBlockChain should have comment or be unexported (golint)
    • Line 53: warning: exported function NewMockBlockChain should have comment or be unexported (golint)
    • Line 60: warning: exported method MockBlockChain.SetFetchContractDataErr should have comment or be unexported (golint)
    • Line 64: warning: exported method MockBlockChain.SetChainHead should have comment or be unexported (golint)
    • Line 68: warning: exported method MockBlockChain.SetChainHeadError should have comment or be unexported (golint)
    • Line 72: warning: exported method MockBlockChain.SetGetEthLogsWithCustomQueryErr should have comment or be unexported (golint)
    • Line 76: warning: exported method MockBlockChain.SetGetEthLogsWithCustomQueryReturnLogs should have comment or be unexported (golint)
    • Line 80: warning: exported method MockBlockChain.FetchContractData should have comment or be unexported (golint)
    • Line 90: warning: exported method MockBlockChain.GetEthLogsWithCustomQuery should have comment or be unexported (golint)
    • Line 95: warning: exported method MockBlockChain.GetHeaderByNumber should have comment or be unexported (golint)
    • Line 99: warning: exported method MockBlockChain.GetHeadersByNumbers should have comment or be unexported (golint)
    • Line 108: warning: exported method MockBlockChain.GetTransactions should have comment or be unexported (golint)
    • Line 114: warning: exported method MockBlockChain.CallContract should have comment or be unexported (golint)
    • Line 118: warning: exported method MockBlockChain.ChainHead should have comment or be unexported (golint)
    • Line 122: warning: exported type BatchGetStorageAtCall should have comment or be unexported (golint)
    • Line 128: warning: exported method MockBlockChain.BatchGetStorageAt should have comment or be unexported (golint)
    • Line 142: warning: exported method MockBlockChain.SetStorageValuesToReturn should have comment or be unexported (golint)
    • Line 150: warning: exported method MockBlockChain.Node should have comment or be unexported (golint)
    • Line 154: warning: exported method MockBlockChain.AssertFetchContractDataCalledWith should have comment or be unexported (golint)
    • Line 165: warning: exported method MockBlockChain.AssertGetEthLogsWithCustomQueryCalledWith should have comment or be unexported (golint)
    • vulcanizedb/pkg/config/contract.go
    • Line 26: warning: comment on exported type ContractConfig should be of the form "ContractConfig ..." (with optional leading article) (golint)
    • Line 56: warning: exported method ContractConfig.PrepConfig should have comment or be unexported (golint)
    • vulcanizedb/pkg/eth/client/subscription.go
    • Line 5: warning: exported type Subscription should have comment or be unexported (golint)
    • Line 9: warning: exported method Subscription.Err should have comment or be unexported (golint)
    • Line 13: warning: exported method Subscription.Unsubscribe should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/watcher/event_watcher.go
    • Line 33: warning: exported type EventWatcher should have comment or be unexported (golint)
    • Line 45: warning: exported function NewEventWatcher should have comment or be unexported (golint)
    • Line 59: warning: comment on exported method EventWatcher.AddTransformers should be of the form "AddTransformers ..." (golint)
    • Line 73: warning: comment on exported method EventWatcher.Execute should be of the form "Execute ..." (golint)
    • vulcanizedb/pkg/fakes/mock_tailer.go
    • Line 24: warning: exported type MockTailer should have comment or be unexported (golint)
    • Line 29: warning: exported function NewMockTailer should have comment or be unexported (golint)
    • Line 35: warning: exported method MockTailer.Tail should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/fetcher/log_fetcher.go
    • Line 27: warning: exported type ILogFetcher should have comment or be unexported (golint)
    • Line 32: warning: exported type LogFetcher should have comment or be unexported (golint)
    • Line 36: warning: exported function NewLogFetcher should have comment or be unexported (golint)
    • Line 42: warning: comment on exported method LogFetcher.FetchLogs should be of the form "FetchLogs ..." (golint)
    • vulcanizedb/libraries/shared/mocks/storage_keys_lookup.go
    • Line 25: warning: exported type MockStorageKeysLookup should have comment or be unexported (golint)
    • Line 35: warning: exported method MockStorageKeysLookup.Lookup should have comment or be unexported (golint)
    • Line 40: warning: exported method MockStorageKeysLookup.GetKeys should have comment or be unexported (golint)
    • Line 45: warning: exported method MockStorageKeysLookup.SetDB should have comment or be unexported (golint)
    • vulcanizedb/utils/utils.go
    • Line 27: warning: exported function LoadPostgres should have comment or be unexported (golint)
    • Line 35: warning: exported function RollbackAndLogFailure should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/log_delegator.go
    • Line 24: warning: exported type MockLogDelegator should have comment or be unexported (golint)
    • Line 31: warning: exported method MockLogDelegator.AddTransformer should have comment or be unexported (golint)
    • Line 35: warning: exported method MockLogDelegator.DelegateLogs should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/storage/types/diff.go
    • Line 28: warning: exported const ExpectedRowLength should have comment or be unexported (golint)
    • Line 30: warning: exported type RawDiff should have comment or be unexported (golint)
    • Line 38: warning: exported type PersistedDiff should have comment or be unexported (golint)
    • Line 47: warning: exported function FromParityCsvRow should have comment or be unexported (golint)
    • Line 64: warning: exported function FromGethStateDiff should have comment or be unexported (golint)
    • Line 80: warning: exported function HexToKeccak256Hash should have comment or be unexported (golint)
    • vulcanizedb/pkg/core/log.go
    • Line 21: warning: exported type ReceiptLog should have comment or be unexported (golint)
    • Line 30: warning: exported type EventLog should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_event_log_repository.go
    • Line 24: warning: exported type MockEventLogRepository should have comment or be unexported (golint)
    • Line 35: warning: exported method MockEventLogRepository.GetUntransformedEventLogs should have comment or be unexported (golint)
    • Line 54: warning: exported method MockEventLogRepository.CreateEventLogs should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/storage_fetcher.go
    • Line 23: warning: exported type MockStorageFetcher should have comment or be unexported (golint)
    • Line 29: warning: exported function NewMockStorageFetcher should have comment or be unexported (golint)
    • Line 33: warning: exported method MockStorageFetcher.FetchStorageDiffs should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/data.go
    • Line 32: warning: exported var FakeAddress should have comment or be unexported (golint)
    • Line 34: warning: error var FakeError should have name of the form ErrFoo (golint)
    • Line 41: warning: exported var FakeHeader should have comment or be unexported (golint)
    • Line 47: warning: exported function GetFakeHeader should have comment or be unexported (golint)
    • Line 51: warning: exported function GetFakeHeaderWithTimestamp should have comment or be unexported (golint)
    • Line 60: warning: exported function RandomString should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/logs/delegator.go
    • Line 32: warning: exported var ErrNoLogs should have comment or be unexported (golint)
    • Line 36: warning: exported type ILogDelegator should have comment or be unexported (golint)
    • Line 41: warning: exported type LogDelegator should have comment or be unexported (golint)
    • Line 47: warning: exported function NewLogDelegator should have comment or be unexported (golint)
    • Line 54: warning: exported method LogDelegator.AddTransformer should have comment or be unexported (golint)
    • Line 59: warning: exported method LogDelegator.DelegateLogs should have comment or be unexported (golint)
    • Line 76: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • vulcanizedb/pkg/fakes/subscription.go
    • Line 3: warning: exported type MockSubscription should have comment or be unexported (golint)
    • Line 7: warning: exported method MockSubscription.Err should have comment or be unexported (golint)
    • Line 11: warning: exported method MockSubscription.Unsubscribe should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/streamer/state_change_streamer.go
    • Line 26: warning: exported type Streamer should have comment or be unexported (golint)
    • Line 30: warning: exported type EthStateChangeStreamer should have comment or be unexported (golint)
    • Line 35: warning: exported function NewEthStateChangeStreamer should have comment or be unexported (golint)
    • Line 42: warning: exported method EthStateChangeStreamer.Stream should have comment or be unexported (golint)
    • vulcanizedb/pkg/eth/client/eth_client.go
    • Line 30: warning: exported type EthClient should have comment or be unexported (golint)
    • Line 34: warning: exported function NewEthClient should have comment or be unexported (golint)
    • Line 38: warning: exported method EthClient.BlockByNumber should have comment or be unexported (golint)
    • Line 42: warning: exported method EthClient.CallContract should have comment or be unexported (golint)
    • Line 46: warning: exported method EthClient.FilterLogs should have comment or be unexported (golint)
    • Line 50: warning: exported method EthClient.HeaderByNumber should have comment or be unexported (golint)
    • Line 54: warning: exported method EthClient.SubscribeNewStateChanges should have comment or be unexported (golint)
    • Line 58: warning: exported method EthClient.TransactionSender should have comment or be unexported (golint)
    • Line 62: warning: exported method EthClient.TransactionReceipt should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_eth_client.go
    • Line 29: warning: should not use dot imports (golint)
    • Line 32: warning: exported type MockEthClient should have comment or be unexported (golint)
    • Line 65: warning: exported function NewMockEthClient should have comment or be unexported (golint)
    • Line 69: warning: exported method MockEthClient.SetCallContractErr should have comment or be unexported (golint)
    • Line 73: warning: exported method MockEthClient.SetCallContractReturnBytes should have comment or be unexported (golint)
    • Line 77: warning: exported method MockEthClient.SetBlockByNumberErr should have comment or be unexported (golint)
    • Line 81: warning: exported method MockEthClient.SetBlockByNumberReturnBlock should have comment or be unexported (golint)
    • Line 85: warning: exported method MockEthClient.SetHeaderByNumberErr should have comment or be unexported (golint)
    • Line 89: warning: exported method MockEthClient.SetHeaderByNumberReturnHeader should have comment or be unexported (golint)
    • Line 93: warning: exported method MockEthClient.SetHeaderByNumbersReturnHeader should have comment or be unexported (golint)
    • Line 97: warning: exported method MockEthClient.SetFilterLogsErr should have comment or be unexported (golint)
    • Line 101: warning: exported method MockEthClient.SetFilterLogsReturnLogs should have comment or be unexported (golint)
    • Line 105: warning: exported method MockEthClient.SetTransactionReceiptErr should have comment or be unexported (golint)
    • Line 109: warning: exported method MockEthClient.SetTransactionReceipts should have comment or be unexported (golint)
    • Line 115: warning: exported method MockEthClient.SetTransactionSenderErr should have comment or be unexported (golint)
    • Line 119: warning: exported method MockEthClient.CallContract should have comment or be unexported (golint)
    • Line 126: warning: exported method MockEthClient.BatchCall should have comment or be unexported (golint)
    • Line 133: warning: exported method MockEthClient.BlockByNumber should have comment or be unexported (golint)
    • Line 139: warning: exported method MockEthClient.HeaderByNumber should have comment or be unexported (golint)
    • Line 145: warning: exported method MockEthClient.HeaderByNumbers should have comment or be unexported (golint)
    • Line 150: warning: exported method MockEthClient.FilterLogs should have comment or be unexported (golint)
    • Line 156: warning: exported method MockEthClient.SubscribeNewStateChanges should have comment or be unexported (golint)
    • Line 163: warning: exported method MockEthClient.TransactionSender should have comment or be unexported (golint)
    • Line 167: warning: exported method MockEthClient.TransactionReceipt should have comment or be unexported (golint)
    • Line 174: warning: exported method MockEthClient.AssertCallContractCalledWith should have comment or be unexported (golint)
    • Line 180: warning: exported method MockEthClient.AssertBlockByNumberCalledWith should have comment or be unexported (golint)
    • Line 185: warning: exported method MockEthClient.AssertHeaderByNumberCalledWith should have comment or be unexported (golint)
    • Line 190: warning: exported method MockEthClient.AssertHeaderByNumbersCalledWith should have comment or be unexported (golint)
    • Line 194: warning: exported method MockEthClient.AssertFilterLogsCalledWith should have comment or be unexported (golint)
    • Line 199: warning: exported method MockEthClient.AssertBatchCalledWith should have comment or be unexported (golint)
    • Line 203: warning: exported method MockEthClient.AssertSubscribeNewStateChangesCalledWith should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/storage/keys_loader.go
    • Line 27: warning: exported const IndexZero should have comment (or a comment on this block) or be unexported (golint)
    • Line 42: warning: exported function GetKeyForMapping should have comment or be unexported (golint)
    • Line 47: warning: exported function GetKeyForNestedMapping should have comment or be unexported (golint)
    • Line 52: warning: exported function GetIncrementedKey should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/factories/storage/transformer.go
    • Line 28: warning: exported type ITransformer should have comment or be unexported (golint)
    • Line 34: warning: exported type TransformerInitializer should have comment or be unexported (golint)
    • Line 36: warning: exported type Transformer should have comment or be unexported (golint)
    • Line 42: warning: exported method Transformer.GetStorageKeysLookup should have comment or be unexported (golint)
    • Line 46: warning: exported method Transformer.GetContractAddress should have comment or be unexported (golint)
    • Line 50: warning: exported method Transformer.NewTransformer should have comment or be unexported (golint)
    • Line 56: warning: exported method Transformer.Execute should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/storage_diff_repository.go
    • Line 23: warning: exported type MockStorageDiffRepository should have comment or be unexported (golint)
    • Line 49: warning: exported method MockStorageDiffRepository.CreateStorageDiff should have comment or be unexported (golint)
    • Line 54: warning: exported method MockStorageDiffRepository.CreateBackFilledStorageValue should have comment or be unexported (golint)
    • Line 59: warning: exported method MockStorageDiffRepository.GetNewDiffs should have comment or be unexported (golint)
    • Line 69: warning: exported method MockStorageDiffRepository.GetUnrecognizedDiffs should have comment or be unexported (golint)
    • Line 79: warning: exported method MockStorageDiffRepository.GetPendingDiffs should have comment or be unexported (golint)
    • Line 89: warning: exported method MockStorageDiffRepository.MarkTransformed should have comment or be unexported (golint)
    • Line 94: warning: exported method MockStorageDiffRepository.MarkNoncanonical should have comment or be unexported (golint)
    • Line 99: warning: exported method MockStorageDiffRepository.MarkNoncanonicalDiffsAsNew should have comment or be unexported (golint)
    • Line 103: warning: exported method MockStorageDiffRepository.MarkUnrecognized should have comment or be unexported (golint)
    • Line 108: warning: exported method MockStorageDiffRepository.MarkUnwatched should have comment or be unexported (golint)
    • Line 113: warning: exported method MockStorageDiffRepository.MarkPending should have comment or be unexported (golint)
    • Line 118: warning: exported method MockStorageDiffRepository.GetFirstDiffIDForBlockHeight should have comment or be unexported (golint)
    • vulcanizedb/pkg/eth/node/node.go
    • Line 29: warning: exported type IPropertiesReader should have comment or be unexported (golint)
    • Line 35: warning: exported type PropertiesReader should have comment or be unexported (golint)
    • Line 39: warning: exported type ParityClient should have comment or be unexported (golint)
    • Line 43: warning: exported type GethClient should have comment or be unexported (golint)
    • Line 47: warning: exported type InfuraClient should have comment or be unexported (golint)
    • Line 51: warning: exported type GanacheClient should have comment or be unexported (golint)
    • Line 55: warning: exported function MakeNode should have comment or be unexported (golint)
    • Line 98: warning: exported method PropertiesReader.NetworkId should have comment or be unexported (golint)
    • Line 108: warning: exported method PropertiesReader.GenesisBlock should have comment or be unexported (golint)
    • Line 119: warning: exported method PropertiesReader.NodeClientName should have comment or be unexported (golint)
    • Line 128: warning: exported method ParityClient.NodeClientName should have comment or be unexported (golint)
    • Line 132: warning: exported method InfuraClient.NodeClientName should have comment or be unexported (golint)
    • Line 136: warning: exported method GanacheClient.NodeClientName should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/block_retriever.go
    • Line 3: warning: exported type MockBlockRetriever should have comment or be unexported (golint)
    • Line 8: warning: exported method MockBlockRetriever.RetrieveFirstBlock should have comment or be unexported (golint)
    • Line 12: warning: exported method MockBlockRetriever.RetrieveMostRecentBlock should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/log_extractor.go
    • Line 25: warning: exported type MockLogExtractor should have comment or be unexported (golint)
    • Line 32: warning: exported method MockLogExtractor.AddTransformerConfig should have comment or be unexported (golint)
    • Line 37: warning: exported method MockLogExtractor.ExtractLogs should have comment or be unexported (golint)
    • Line 52: warning: exported method MockLogExtractor.BackFillLogs should have comment or be unexported (golint)
    • vulcanizedb/pkg/eth/abi.go
    • Line 33: warning: exported var ErrInvalidAbiFile should have comment or be unexported (golint)
    • Line 38: warning: exported type Response should have comment or be unexported (golint)
    • Line 44: warning: exported type EtherScanAPI should have comment or be unexported (golint)
    • Line 49: warning: exported function NewEtherScanClient should have comment or be unexported (golint)
    • Line 56: warning: exported function GenURL should have comment or be unexported (golint)
    • Line 69: warning: comment on exported method EtherScanAPI.GetAbi should be of the form "GetAbi ..." (golint)
    • Line 87: warning: exported function ParseAbiFile should have comment or be unexported (golint)
    • Line 95: warning: exported function ParseAbi should have comment or be unexported (golint)
    • Line 103: warning: exported function ReadAbiFile should have comment or be unexported (golint)
    • vulcanizedb/pkg/eth/blockchain.go
    • Line 33: warning: exported var ErrEmptyHeader should have comment or be unexported (golint)
    • Line 35: warning: exported const MAX_BATCH_SIZE should have comment or be unexported (golint)
    • Line 37: warning: exported type BlockChain should have comment or be unexported (golint)
    • Line 45: warning: exported function NewBlockChain should have comment or be unexported (golint)
    • Line 55: warning: exported method BlockChain.GetEthLogsWithCustomQuery should have comment or be unexported (golint)
    • Line 63: warning: exported method BlockChain.GetHeaderByNumber should have comment or be unexported (golint)
    • Line 70: warning: exported method BlockChain.GetHeadersByNumbers should have comment or be unexported (golint)
    • Line 77: warning: exported method BlockChain.GetTransactions should have comment or be unexported (golint)
    • Line 99: warning: exported method BlockChain.ChainHead should have comment or be unexported (golint)
    • Line 107: warning: exported method BlockChain.BatchGetStorageAt should have comment or be unexported (golint)
    • Line 134: warning: exported method BlockChain.Node should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/test_data/generic.go
    • Line 17: warning: don't use an underscore in package name (golint)
    • Line 32: warning: exported var GenericTestLog should have comment or be unexported (golint)
    • Line 45: warning: exported var GenericTestConfig should have comment or be unexported (golint)
    • Line 54: warning: exported function FakeAddress should have comment or be unexported (golint)
    • Line 58: warning: exported function FakeHash should have comment or be unexported (golint)
    • vulcanizedb/pkg/plugin/writer/writer.go
    • Line 23: warning: should not use dot imports (golint)
    • Line 29: warning: comment on exported type PluginWriter should be of the form "PluginWriter ..." (with optional leading article) (golint)
    • Line 40: warning: comment on exported function NewPluginWriter should be of the form "NewPluginWriter ..." (golint)
    • Line 94: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 112: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • vulcanizedb/pkg/fakes/checked_logs_repository.go
    • Line 19: warning: exported type MockCheckedLogsRepository should have comment or be unexported (golint)
    • Line 29: warning: exported method MockCheckedLogsRepository.AlreadyWatchingLog should have comment or be unexported (golint)
    • Line 35: warning: exported method MockCheckedLogsRepository.MarkLogWatched should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/watcher/storage_watcher.go
    • Line 37: warning: exported var ErrHeaderMismatch should have comment or be unexported (golint)
    • Line 42: warning: exported type IStorageWatcher should have comment or be unexported (golint)
    • Line 48: warning: exported type StorageWatcher should have comment or be unexported (golint)
    • Line 60: warning: exported type DiffStatusToWatch should have comment or be unexported (golint)
    • Line 63: warning: exported const New should have comment (or a comment on this block) or be unexported (golint)
    • Line 68: warning: exported function NewStorageWatcher should have comment or be unexported (golint)
    • Line 72: warning: exported function UnrecognizedStorageWatcher should have comment or be unexported (golint)
    • Line 76: warning: exported function PendingStorageWatcher should have comment or be unexported (golint)
    • Line 99: warning: exported method StorageWatcher.StorageWatcherName should have comment or be unexported (golint)
    • Line 111: warning: exported method StorageWatcher.AddTransformers should have comment or be unexported (golint)
    • Line 118: warning: exported method StorageWatcher.Execute should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/watcher/throttler.go
    • Line 5: warning: exported type Callback should have comment or be unexported (golint)
    • Line 6: warning: exported type ThrottlerFunc should have comment or be unexported (golint)
    • Line 8: warning: exported type Timer should have comment or be unexported (golint)
    • Line 14: warning: exported type StandardTimer should have comment or be unexported (golint)
    • Line 18: warning: exported method StandardTimer.WaitFor should have comment or be unexported (golint)
    • Line 22: warning: exported method StandardTimer.ElapsedTime should have comment or be unexported (golint)
    • Line 27: warning: exported method StandardTimer.Start should have comment or be unexported (golint)
    • Line 31: warning: exported type Throttler should have comment or be unexported (golint)
    • Line 35: warning: exported function NewThrottler should have comment or be unexported (golint)
    • Line 41: warning: exported method Throttler.Throttle should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/logs/extractor.go
    • Line 35: warning: exported type BlockIdentifier should have comment or be unexported (golint)
    • Line 38: warning: exported var StartInterval should have comment or be unexported (golint)
    • Line 45: warning: exported type ILogExtractor should have comment or be unexported (golint)
    • Line 51: warning: exported type LogExtractor should have comment or be unexported (golint)
    • Line 65: warning: exported function NewLogExtractor should have comment or be unexported (golint)
    • Line 185: warning: exported function ChunkRanges should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/storage/types/value.go
    • Line 21: warning: exported type ValueType should have comment or be unexported (golint)
    • Line 24: warning: exported const Uint256 should have comment (or a comment on this block) or be unexported (golint)
    • Line 34: warning: exported type Key should have comment or be unexported (golint)
    • Line 36: warning: exported type ValueMetadata should have comment or be unexported (golint)
    • Line 44: warning: exported function GetValueMetadata should have comment or be unexported (golint)
    • Line 48: warning: exported function GetValueMetadataForPackedSlot should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_transaction_converter.go
    • Line 23: warning: exported type MockTransactionConverter should have comment or be unexported (golint)
    • Line 27: warning: exported function NewMockTransactionConverter should have comment or be unexported (golint)
    • Line 33: warning: exported method MockTransactionConverter.ConvertRpcTransactionsToModels should have comment or be unexported (golint)
    • vulcanizedb/pkg/fs/tail.go
    • Line 5: warning: exported type Tailer should have comment or be unexported (golint)
    • Line 9: warning: exported type FileTailer should have comment or be unexported (golint)
    • Line 13: warning: exported method FileTailer.Tail should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/event_transformer.go
    • Line 26: warning: exported type MockEventTransformer should have comment or be unexported (golint)
    • Line 33: warning: exported method MockEventTransformer.Execute should have comment or be unexported (golint)
    • Line 42: warning: exported method MockEventTransformer.GetConfig should have comment or be unexported (golint)
    • Line 46: warning: exported method MockEventTransformer.SetTransformerConfig should have comment or be unexported (golint)
    • Line 50: warning: exported method MockEventTransformer.FakeTransformerInitializer should have comment or be unexported (golint)
    • Line 54: warning: exported var FakeTransformerConfig should have comment or be unexported (golint)
    • vulcanizedb/pkg/history/header_validator.go
    • Line 26: warning: exported type HeaderValidator should have comment or be unexported (golint)
    • Line 32: warning: exported function NewHeaderValidator should have comment or be unexported (golint)
    • Line 40: warning: exported method HeaderValidator.ValidateHeaders should have comment or be unexported (golint)
    • vulcanizedb/pkg/fakes/mock_header_repository.go
    • Line 22: warning: should not use dot imports (golint)
    • Line 25: warning: exported type MockHeaderRepository should have comment or be unexported (golint)
    • Line 50: warning: exported function NewMockHeaderRepository should have comment or be unexported (golint)
    • Line 54: warning: exported method MockHeaderRepository.SetCreateOrUpdateHeaderReturnID should have comment or be unexported (golint)
    • Line 58: warning: exported method MockHeaderRepository.SetCreateOrUpdateHeaderReturnErr should have comment or be unexported (golint)
    • Line 62: warning: exported method MockHeaderRepository.SetMissingBlockNumbers should have comment or be unexported (golint)
    • Line 66: warning: exported method MockHeaderRepository.CreateOrUpdateHeader should have comment or be unexported (golint)
    • Line 72: warning: exported method MockHeaderRepository.CreateTransactions should have comment or be unexported (golint)
    • Line 77: warning: exported method MockHeaderRepository.CreateTransactionInTx should have comment or be unexported (golint)
    • Line 81: warning: exported method MockHeaderRepository.DeleteHeader should have comment or be unexported (golint)
    • Line 85: warning: exported method MockHeaderRepository.GetHeaderByBlockNumber should have comment or be unexported (golint)
    • Line 94: warning: exported method MockHeaderRepository.GetHeaderByID should have comment or be unexported (golint)
    • Line 98: warning: exported method MockHeaderRepository.GetHeadersInRange should have comment or be unexported (golint)
    • Line 104: warning: exported method MockHeaderRepository.MissingBlockNumbers should have comment or be unexported (golint)
    • Line 110: warning: exported method MockHeaderRepository.GetMostRecentHeaderBlockNumber should have comment or be unexported (golint)
    • Line 114: warning: exported method MockHeaderRepository.AssertCreateOrUpdateHeaderCallCountAndPassedBlockNumbers should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/mocks/storage_transformer.go
    • Line 26: warning: exported type MockStorageTransformer should have comment or be unexported (golint)
    • Line 33: warning: exported method MockStorageTransformer.Execute should have comment or be unexported (golint)
    • Line 38: warning: exported method MockStorageTransformer.GetContractAddress should have comment or be unexported (golint)
    • Line 42: warning: exported method MockStorageTransformer.GetStorageKeysLookup should have comment or be unexported (golint)
    • Line 46: warning: exported method MockStorageTransformer.FakeTransformerInitializer should have comment or be unexported (golint)
    • vulcanizedb/libraries/shared/storage/types/errors.go
    • Line 24: warning: exported type ErrContractNotFound should have comment or be unexported (golint)
    • Line 32: warning: exported type ErrMetadataMalformed should have comment or be unexported (golint)
    • Line 40: warning: exported type ErrRowMalformed should have comment or be unexported (golint)
    • Line 48: warning: exported var ErrKeyNotFound should have comment or be unexported (golint)
    • vulcanizedb/pkg/datastore/postgres/errors.go
    • Line 11: warning: exported const DbConnectionFailedMsg should have comment (or a comment on this block) or be unexported (golint)
    • Line 16: warning: exported var ErrHeaderDoesNotExist should have comment or be unexported (golint)
    • Line 18: warning: exported function ErrDBConnectionFailed should have comment or be unexported (golint)
    • Line 22: warning: exported function ErrUnableToSetNode should have comment or be unexported (golint)
    • Line 27: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 30: warning: exported function UnwrapErrorRecursively should have comment or be unexported (golint)
    • Line 38: warning: exported function IsForeignKeyViolationErr should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!