Preparing report...

Report for github.com/9corp/9volt

A+    Excellent!    Found 45 issues across 70 files

Tweet

gofmt95%

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!


gocyclo100%

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.

No problems detected. Good job!


golint45%

Golint is a linter for Go source code.

    • 9volt/api/state.go
    • Line 10: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 13: warning: comment on exported method Api.StateHandler should be of the form "StateHandler ..." (golint)
    • Line 34: warning: exported method Api.StateWithTagsHandler should have comment or be unexported (golint)
    • 9volt/fakes/dalfakes/fake_dal.go
    • Line 1: warning: package comment should be of the form "Package dalfakes ..." (golint)
    • Line 11: warning: exported type FakeIDal should have comment or be unexported (golint)
    • Line 374: warning: exported method FakeIDal.Get should have comment or be unexported (golint)
    • Line 392: warning: exported method FakeIDal.GetCallCount should have comment or be unexported (golint)
    • Line 398: warning: exported method FakeIDal.GetArgsForCall should have comment or be unexported (golint)
    • Line 404: warning: exported method FakeIDal.GetReturns should have comment or be unexported (golint)
    • Line 412: warning: exported method FakeIDal.GetReturnsOnCall should have comment or be unexported (golint)
    • Line 426: warning: exported method FakeIDal.Set should have comment or be unexported (golint)
    • Line 445: warning: exported method FakeIDal.SetCallCount should have comment or be unexported (golint)
    • Line 451: warning: exported method FakeIDal.SetArgsForCall should have comment or be unexported (golint)
    • Line 457: warning: exported method FakeIDal.SetReturns should have comment or be unexported (golint)
    • Line 464: warning: exported method FakeIDal.SetReturnsOnCall should have comment or be unexported (golint)
    • Line 476: warning: exported method FakeIDal.Delete should have comment or be unexported (golint)
    • Line 494: warning: exported method FakeIDal.DeleteCallCount should have comment or be unexported (golint)
    • Line 500: warning: exported method FakeIDal.DeleteArgsForCall should have comment or be unexported (golint)
    • Line 506: warning: exported method FakeIDal.DeleteReturns should have comment or be unexported (golint)
    • Line 513: warning: exported method FakeIDal.DeleteReturnsOnCall should have comment or be unexported (golint)
    • Line 525: warning: exported method FakeIDal.Refresh should have comment or be unexported (golint)
    • Line 543: warning: exported method FakeIDal.RefreshCallCount should have comment or be unexported (golint)
    • Line 549: warning: exported method FakeIDal.RefreshArgsForCall should have comment or be unexported (golint)
    • Line 555: warning: exported method FakeIDal.RefreshReturns should have comment or be unexported (golint)
    • Line 562: warning: exported method FakeIDal.RefreshReturnsOnCall should have comment or be unexported (golint)
    • Line 574: warning: exported method FakeIDal.KeyExists should have comment or be unexported (golint)
    • Line 591: warning: exported method FakeIDal.KeyExistsCallCount should have comment or be unexported (golint)
    • Line 597: warning: exported method FakeIDal.KeyExistsArgsForCall should have comment or be unexported (golint)
    • Line 603: warning: exported method FakeIDal.KeyExistsReturns should have comment or be unexported (golint)
    • Line 612: warning: exported method FakeIDal.KeyExistsReturnsOnCall should have comment or be unexported (golint)
    • Line 628: warning: exported method FakeIDal.IsKeyNotFound should have comment or be unexported (golint)
    • Line 645: warning: exported method FakeIDal.IsKeyNotFoundCallCount should have comment or be unexported (golint)
    • Line 651: warning: exported method FakeIDal.IsKeyNotFoundArgsForCall should have comment or be unexported (golint)
    • Line 657: warning: exported method FakeIDal.IsKeyNotFoundReturns should have comment or be unexported (golint)
    • Line 664: warning: exported method FakeIDal.IsKeyNotFoundReturnsOnCall should have comment or be unexported (golint)
    • Line 676: warning: exported method FakeIDal.CreateDirectorState should have comment or be unexported (golint)
    • Line 693: warning: exported method FakeIDal.CreateDirectorStateCallCount should have comment or be unexported (golint)
    • Line 699: warning: exported method FakeIDal.CreateDirectorStateArgsForCall should have comment or be unexported (golint)
    • Line 705: warning: exported method FakeIDal.CreateDirectorStateReturns should have comment or be unexported (golint)
    • Line 712: warning: exported method FakeIDal.CreateDirectorStateReturnsOnCall should have comment or be unexported (golint)
    • Line 724: warning: exported method FakeIDal.UpdateDirectorState should have comment or be unexported (golint)
    • Line 743: warning: exported method FakeIDal.UpdateDirectorStateCallCount should have comment or be unexported (golint)
    • Line 749: warning: exported method FakeIDal.UpdateDirectorStateArgsForCall should have comment or be unexported (golint)
    • Line 755: warning: exported method FakeIDal.UpdateDirectorStateReturns should have comment or be unexported (golint)
    • Line 762: warning: exported method FakeIDal.UpdateDirectorStateReturnsOnCall should have comment or be unexported (golint)
    • Line 774: warning: exported method FakeIDal.NewWatcher should have comment or be unexported (golint)
    • Line 792: warning: exported method FakeIDal.NewWatcherCallCount should have comment or be unexported (golint)
    • Line 798: warning: exported method FakeIDal.NewWatcherArgsForCall should have comment or be unexported (golint)
    • Line 804: warning: exported method FakeIDal.NewWatcherReturns should have comment or be unexported (golint)
    • Line 811: warning: exported method FakeIDal.NewWatcherReturnsOnCall should have comment or be unexported (golint)
    • Line 823: warning: exported method FakeIDal.NewWatcherForOverwatch should have comment or be unexported (golint)
    • Line 841: warning: exported method FakeIDal.NewWatcherForOverwatchCallCount should have comment or be unexported (golint)
    • Line 847: warning: exported method FakeIDal.NewWatcherForOverwatchArgsForCall should have comment or be unexported (golint)
    • Line 853: warning: exported method FakeIDal.NewWatcherForOverwatchReturns should have comment or be unexported (golint)
    • Line 861: warning: exported method FakeIDal.NewWatcherForOverwatchReturnsOnCall should have comment or be unexported (golint)
    • Line 875: warning: exported method FakeIDal.GetClusterMembers should have comment or be unexported (golint)
    • Line 890: warning: exported method FakeIDal.GetClusterMembersCallCount should have comment or be unexported (golint)
    • Line 896: warning: exported method FakeIDal.GetClusterMembersReturns should have comment or be unexported (golint)
    • Line 904: warning: exported method FakeIDal.GetClusterMembersReturnsOnCall should have comment or be unexported (golint)
    • Line 918: warning: exported method FakeIDal.GetCheckKeys should have comment or be unexported (golint)
    • Line 933: warning: exported method FakeIDal.GetCheckKeysCallCount should have comment or be unexported (golint)
    • Line 939: warning: exported method FakeIDal.GetCheckKeysReturns should have comment or be unexported (golint)
    • Line 947: warning: exported method FakeIDal.GetCheckKeysReturnsOnCall should have comment or be unexported (golint)
    • Line 961: warning: exported method FakeIDal.GetCheckKeysWithMemberTag should have comment or be unexported (golint)
    • Line 976: warning: exported method FakeIDal.GetCheckKeysWithMemberTagCallCount should have comment or be unexported (golint)
    • Line 982: warning: exported method FakeIDal.GetCheckKeysWithMemberTagReturns should have comment or be unexported (golint)
    • Line 990: warning: exported method FakeIDal.GetCheckKeysWithMemberTagReturnsOnCall should have comment or be unexported (golint)
    • Line 1004: warning: exported method FakeIDal.CreateCheckReference should have comment or be unexported (golint)
    • Line 1022: warning: exported method FakeIDal.CreateCheckReferenceCallCount should have comment or be unexported (golint)
    • Line 1028: warning: exported method FakeIDal.CreateCheckReferenceArgsForCall should have comment or be unexported (golint)
    • Line 1034: warning: exported method FakeIDal.CreateCheckReferenceReturns should have comment or be unexported (golint)
    • Line 1041: warning: exported method FakeIDal.CreateCheckReferenceReturnsOnCall should have comment or be unexported (golint)
    • Line 1053: warning: exported method FakeIDal.ClearCheckReference should have comment or be unexported (golint)
    • Line 1071: warning: exported method FakeIDal.ClearCheckReferenceCallCount should have comment or be unexported (golint)
    • Line 1077: warning: exported method FakeIDal.ClearCheckReferenceArgsForCall should have comment or be unexported (golint)
    • Line 1083: warning: exported method FakeIDal.ClearCheckReferenceReturns should have comment or be unexported (golint)
    • Line 1090: warning: exported method FakeIDal.ClearCheckReferenceReturnsOnCall should have comment or be unexported (golint)
    • Line 1102: warning: exported method FakeIDal.ClearCheckReferences should have comment or be unexported (golint)
    • Line 1119: warning: exported method FakeIDal.ClearCheckReferencesCallCount should have comment or be unexported (golint)
    • Line 1125: warning: exported method FakeIDal.ClearCheckReferencesArgsForCall should have comment or be unexported (golint)
    • Line 1131: warning: exported method FakeIDal.ClearCheckReferencesReturns should have comment or be unexported (golint)
    • Line 1138: warning: exported method FakeIDal.ClearCheckReferencesReturnsOnCall should have comment or be unexported (golint)
    • Line 1150: warning: exported method FakeIDal.FetchAllMemberRefs should have comment or be unexported (golint)
    • Line 1165: warning: exported method FakeIDal.FetchAllMemberRefsCallCount should have comment or be unexported (golint)
    • Line 1171: warning: exported method FakeIDal.FetchAllMemberRefsReturns should have comment or be unexported (golint)
    • Line 1180: warning: exported method FakeIDal.FetchAllMemberRefsReturnsOnCall should have comment or be unexported (golint)
    • Line 1196: warning: exported method FakeIDal.FetchCheckStats should have comment or be unexported (golint)
    • Line 1211: warning: exported method FakeIDal.FetchCheckStatsCallCount should have comment or be unexported (golint)
    • Line 1217: warning: exported method FakeIDal.FetchCheckStatsReturns should have comment or be unexported (golint)
    • Line 1225: warning: exported method FakeIDal.FetchCheckStatsReturnsOnCall should have comment or be unexported (golint)
    • Line 1239: warning: exported method FakeIDal.FetchAlerterConfig should have comment or be unexported (golint)
    • Line 1256: warning: exported method FakeIDal.FetchAlerterConfigCallCount should have comment or be unexported (golint)
    • Line 1262: warning: exported method FakeIDal.FetchAlerterConfigArgsForCall should have comment or be unexported (golint)
    • Line 1268: warning: exported method FakeIDal.FetchAlerterConfigReturns should have comment or be unexported (golint)
    • Line 1276: warning: exported method FakeIDal.FetchAlerterConfigReturnsOnCall should have comment or be unexported (golint)
    • Line 1290: warning: exported method FakeIDal.FetchState should have comment or be unexported (golint)
    • Line 1305: warning: exported method FakeIDal.FetchStateCallCount should have comment or be unexported (golint)
    • Line 1311: warning: exported method FakeIDal.FetchStateReturns should have comment or be unexported (golint)
    • Line 1319: warning: exported method FakeIDal.FetchStateReturnsOnCall should have comment or be unexported (golint)
    • Line 1333: warning: exported method FakeIDal.FetchStateWithTags should have comment or be unexported (golint)
    • Line 1355: warning: exported method FakeIDal.FetchStateWithTagsCallCount should have comment or be unexported (golint)
    • Line 1361: warning: exported method FakeIDal.FetchStateWithTagsArgsForCall should have comment or be unexported (golint)
    • Line 1367: warning: exported method FakeIDal.FetchStateWithTagsReturns should have comment or be unexported (golint)
    • Line 1375: warning: exported method FakeIDal.FetchStateWithTagsReturnsOnCall should have comment or be unexported (golint)
    • Line 1389: warning: exported method FakeIDal.UpdateCheckState should have comment or be unexported (golint)
    • Line 1407: warning: exported method FakeIDal.UpdateCheckStateCallCount should have comment or be unexported (golint)
    • Line 1413: warning: exported method FakeIDal.UpdateCheckStateArgsForCall should have comment or be unexported (golint)
    • Line 1419: warning: exported method FakeIDal.UpdateCheckStateReturns should have comment or be unexported (golint)
    • Line 1426: warning: exported method FakeIDal.UpdateCheckStateReturnsOnCall should have comment or be unexported (golint)
    • Line 1438: warning: exported method FakeIDal.GetClusterStats should have comment or be unexported (golint)
    • Line 1453: warning: exported method FakeIDal.GetClusterStatsCallCount should have comment or be unexported (golint)
    • Line 1459: warning: exported method FakeIDal.GetClusterStatsReturns should have comment or be unexported (golint)
    • Line 1467: warning: exported method FakeIDal.GetClusterStatsReturnsOnCall should have comment or be unexported (golint)
    • Line 1481: warning: exported method FakeIDal.FetchEvents should have comment or be unexported (golint)
    • Line 1503: warning: exported method FakeIDal.FetchEventsCallCount should have comment or be unexported (golint)
    • Line 1509: warning: exported method FakeIDal.FetchEventsArgsForCall should have comment or be unexported (golint)
    • Line 1515: warning: exported method FakeIDal.FetchEventsReturns should have comment or be unexported (golint)
    • Line 1523: warning: exported method FakeIDal.FetchEventsReturnsOnCall should have comment or be unexported (golint)
    • Line 1537: warning: exported method FakeIDal.GetClusterMembersWithTags should have comment or be unexported (golint)
    • Line 1552: warning: exported method FakeIDal.GetClusterMembersWithTagsCallCount should have comment or be unexported (golint)
    • Line 1558: warning: exported method FakeIDal.GetClusterMembersWithTagsReturns should have comment or be unexported (golint)
    • Line 1566: warning: exported method FakeIDal.GetClusterMembersWithTagsReturnsOnCall should have comment or be unexported (golint)
    • Line 1580: warning: exported method FakeIDal.GetClusterMemberTags should have comment or be unexported (golint)
    • Line 1597: warning: exported method FakeIDal.GetClusterMemberTagsCallCount should have comment or be unexported (golint)
    • Line 1603: warning: exported method FakeIDal.GetClusterMemberTagsArgsForCall should have comment or be unexported (golint)
    • Line 1609: warning: exported method FakeIDal.GetClusterMemberTagsReturns should have comment or be unexported (golint)
    • Line 1617: warning: exported method FakeIDal.GetClusterMemberTagsReturnsOnCall should have comment or be unexported (golint)
    • Line 1631: warning: exported method FakeIDal.GetCheckMemberTag should have comment or be unexported (golint)
    • Line 1648: warning: exported method FakeIDal.GetCheckMemberTagCallCount should have comment or be unexported (golint)
    • Line 1654: warning: exported method FakeIDal.GetCheckMemberTagArgsForCall should have comment or be unexported (golint)
    • Line 1660: warning: exported method FakeIDal.GetCheckMemberTagReturns should have comment or be unexported (golint)
    • Line 1668: warning: exported method FakeIDal.GetCheckMemberTagReturnsOnCall should have comment or be unexported (golint)
    • Line 1682: warning: exported method FakeIDal.PushConfigs should have comment or be unexported (golint)
    • Line 1700: warning: exported method FakeIDal.PushConfigsCallCount should have comment or be unexported (golint)
    • Line 1706: warning: exported method FakeIDal.PushConfigsArgsForCall should have comment or be unexported (golint)
    • Line 1712: warning: exported method FakeIDal.PushConfigsReturns should have comment or be unexported (golint)
    • Line 1721: warning: exported method FakeIDal.PushConfigsReturnsOnCall should have comment or be unexported (golint)
    • Line 1737: warning: exported method FakeIDal.PushFullConfigs should have comment or be unexported (golint)
    • Line 1754: warning: exported method FakeIDal.PushFullConfigsCallCount should have comment or be unexported (golint)
    • Line 1760: warning: exported method FakeIDal.PushFullConfigsArgsForCall should have comment or be unexported (golint)
    • Line 1766: warning: exported method FakeIDal.PushFullConfigsReturns should have comment or be unexported (golint)
    • Line 1774: warning: exported method FakeIDal.PushFullConfigsReturnsOnCall should have comment or be unexported (golint)
    • Line 1788: warning: exported method FakeIDal.Invocations should have comment or be unexported (golint)
    • 9volt/dal/dal.go
    • Line 1: warning: package comment should be of the form "Package dal ..." (golint)
    • Line 23: warning: exported type IDal should have comment or be unexported (golint)
    • Line 55: warning: exported type GetOptions should have comment or be unexported (golint)
    • Line 66: warning: exported type Dal should have comment or be unexported (golint)
    • Line 78: warning: comment on exported type MemberStat should be of the form "MemberStat ..." (with optional leading article) (golint)
    • Line 84: warning: exported type FullConfigs should have comment or be unexported (golint)
    • Line 89: warning: exported function New should have comment or be unexported (golint)
    • Line 129: warning: comment on exported method Dal.KeyExists should be of the form "KeyExists ..." (golint)
    • Line 144: warning: exported type SetOptions should have comment or be unexported (golint)
    • Line 162: warning: comment on exported method Dal.Set should be of the form "Set ..." (golint)
    • Line 263: warning: comment on exported method Dal.Refresh should be of the form "Refresh ..." (golint)
    • Line 326: warning: comment on exported method Dal.CreateCheckReference should be of the form "CreateCheckReference ..." (golint)
    • Line 343: warning: comment on exported method Dal.FetchAllMemberRefs should be of the form "FetchAllMemberRefs ..." (golint)
    • Line 397: warning: comment on exported method Dal.CreateDirectorState should be of the form "CreateDirectorState ..." (golint)
    • Line 411: warning: exported method Dal.Delete should have comment or be unexported (golint)
    • Line 423: warning: exported method Dal.NewWatcher should have comment or be unexported (golint)
    • Line 429: warning: comment on exported method Dal.NewWatcherForOverwatch should be of the form "NewWatcherForOverwatch ..." (golint)
    • Line 441: warning: comment on exported method Dal.UpdateDirectorState should be of the form "UpdateDirectorState ..." (golint)
    • Line 460: warning: comment on exported method Dal.ClearCheckReference should be of the form "ClearCheckReference ..." (golint)
    • Line 473: warning: comment on exported method Dal.ClearCheckReferences should be of the form "ClearCheckReferences ..." (golint)
    • Line 487: warning: should omit 2nd value from range; this loop is equivalent to `for key := range ...` (golint)
    • Line 504: warning: comment on exported method Dal.GetClusterMembers should be of the form "GetClusterMembers ..." (golint)
    • Line 523: warning: comment on exported method Dal.GetClusterMembersWithTags should be of the form "GetClusterMembersWithTags ..." (golint)
    • Line 580: warning: comment on exported method Dal.FetchCheckStats should be of the form "FetchCheckStats ..." (golint)
    • Line 625: warning: comment on exported method Dal.GetCheckKeys should be of the form "GetCheckKeys ..." (golint)
    • Line 644: warning: exported method Dal.GetCheckMemberTag should have comment or be unexported (golint)
    • Line 660: warning: comment on exported method Dal.GetCheckKeysWithMemberTag should be of the form "GetCheckKeysWithMemberTag ..." (golint)
    • Line 685: warning: comment on exported method Dal.GetClusterMemberTags should be of the form "GetClusterMemberTags ..." (golint)
    • Line 705: warning: comment on exported method Dal.FetchAlerterConfig should be of the form "FetchAlerterConfig ..." (golint)
    • Line 715: warning: comment on exported method Dal.IsKeyNotFound should be of the form "IsKeyNotFound ..." (golint)
    • 9volt/api/monitor.go
    • Line 21: warning: comment on exported method Api.MonitorHandler should be of the form "MonitorHandler ..." (golint)
    • Line 61: warning: comment on exported method Api.MonitorDisableHandler should be of the form "MonitorDisableHandler ..." (golint)
    • Line 103: warning: comment on exported method Api.MonitorAddHandler should be of the form "MonitorAddHandler ..." (golint)
    • Line 144: warning: comment on exported method Api.MonitorDeleteHandler should be of the form "MonitorDeleteHandler ..." (golint)
    • Line 176: warning: exported method Api.MonitorCheckHandler should have comment or be unexported (golint)
    • 9volt/alerter/alerter.go
    • Line 1: warning: package comment should be of the form "Package alerter ..." (golint)
    • Line 26: warning: exported type IAlerter should have comment or be unexported (golint)
    • Line 32: warning: exported type AlerterConfig should have comment or be unexported (golint)
    • Line 38: warning: exported type Alerter should have comment or be unexported (golint)
    • Line 48: warning: exported type Message should have comment or be unexported (golint)
    • Line 60: warning: exported function New should have comment or be unexported (golint)
    • Line 72: warning: exported method Alerter.Start should have comment or be unexported (golint)
    • Line 94: warning: exported method Alerter.Stop should have comment or be unexported (golint)
    • 9volt/monitor/monitor.go
    • Line 21: warning: exported const START should have comment (or a comment on this block) or be unexported (golint)
    • Line 28: warning: exported type IMonitor should have comment or be unexported (golint)
    • Line 35: warning: exported type Monitor should have comment or be unexported (golint)
    • Line 47: warning: exported type RootMonitorConfig should have comment or be unexported (golint)
    • Line 60: warning: comment on exported type MonitorConfig should be of the form "MonitorConfig ..." (with optional leading article) (golint)
    • Line 105: warning: exported type Response should have comment or be unexported (golint)
    • Line 107: warning: exported function New should have comment or be unexported (golint)
    • Line 126: warning: comment on exported method Monitor.Handle should be of the form "Handle ..." (golint)
    • Line 208: warning: exported method Monitor.StopAll should have comment or be unexported (golint)
    • 9volt/main.go
    • Line 28: warning: exported const DEFAULT_VERSION should have comment (or a comment on this block) or be unexported (golint)
    • 9volt/director/director.go
    • Line 22: warning: exported const COLLECT_CHECK_STATS_INTERVAL should have comment (or a comment on this block) or be unexported (golint)
    • Line 26: warning: exported type IDirector should have comment or be unexported (golint)
    • Line 30: warning: exported type Director should have comment or be unexported (golint)
    • Line 48: warning: exported function New should have comment or be unexported (golint)
    • Line 72: warning: exported method Director.Start should have comment or be unexported (golint)
    • Line 85: warning: exported method Director.Stop should have comment or be unexported (golint)
    • Line 562: warning: comment on exported method Director.PickNextMember should be of the form "PickNextMember ..." (golint)
    • 9volt/api/common.go
    • Line 10: warning: exported method Api.HomeHandler should have comment or be unexported (golint)
    • Line 15: warning: exported method Api.StatusHandler should have comment or be unexported (golint)
    • Line 28: warning: exported method Api.VersionHandler should have comment or be unexported (golint)
    • 9volt/util/util.go
    • Line 19: warning: exported type CustomDuration should have comment or be unexported (golint)
    • Line 21: warning: exported method CustomDuration.UnmarshalJSON should have comment or be unexported (golint)
    • Line 48: warning: exported method CustomDuration.MarshalJSON should have comment or be unexported (golint)
    • Line 56: warning: exported function MD5Hash should have comment or be unexported (golint)
    • Line 68: warning: comment on exported function RandomString should be of the form "RandomString ..." (golint)
    • Line 84: warning: exported function GetMemberID should have comment or be unexported (golint)
    • Line 89: warning: exported function StringSliceContains should have comment or be unexported (golint)
    • Line 99: warning: exported function SplitTags should have comment or be unexported (golint)
    • Line 109: warning: comment on exported function StringSliceInStringSlice should be of the form "StringSliceInStringSlice ..." (golint)
    • Line 122: warning: comment on exported function GetMapKeys should be of the form "GetMapKeys ..." (golint)
    • 9volt/dal/api_cluster.go
    • Line 8: warning: exported type ClusterStats should have comment or be unexported (golint)
    • Line 13: warning: exported method Dal.GetClusterStats should have comment or be unexported (golint)
    • 9volt/monitor/tcp_monitor.go
    • Line 13: warning: exported const DEFAULT_CONN_TIMEOUT should have comment (or a comment on this block) or be unexported (golint)
    • Line 19: warning: exported type TCPMonitor should have comment or be unexported (golint)
    • Line 28: warning: exported function NewTCPMonitor should have comment or be unexported (golint)
    • Line 43: warning: exported method TCPMonitor.Validate should have comment or be unexported (golint)
    • Line 139: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • 9volt/config/config.go
    • Line 14: warning: exported const DEFAULT_CONFIG should have comment (or a comment on this block) or be unexported (golint)
    • Line 17: warning: exported type Config should have comment or be unexported (golint)
    • Line 34: warning: exported type Health should have comment or be unexported (golint)
    • Line 46: warning: comment on exported function New should be of the form "New ..." (golint)
    • Line 77: warning: exported method Config.ValidateDirs should have comment or be unexported (golint)
    • Line 107: warning: exported method Config.Load should have comment or be unexported (golint)
    • 9volt/state/state.go
    • Line 1: warning: package comment should be of the form "Package state ..." (golint)
    • Line 21: warning: exported const STATE_PREFIX should have comment (or a comment on this block) or be unexported (golint)
    • Line 26: warning: exported type Message should have comment or be unexported (golint)
    • Line 36: warning: exported type State should have comment or be unexported (golint)
    • Line 47: warning: comment on exported type TmpMonitorConfig should be of the form "TmpMonitorConfig ..." (with optional leading article) (golint)
    • Line 52: warning: exported function New should have comment or be unexported (golint)
    • Line 66: warning: exported method State.Start should have comment or be unexported (golint)
    • Line 77: warning: exported method State.Stop should have comment or be unexported (golint)
    • 9volt/alerter/slack_alerter.go
    • Line 15: warning: exported const RESOLVE_COLOR should have comment (or a comment on this block) or be unexported (golint)
    • Line 22: warning: exported type Slack should have comment or be unexported (golint)
    • Line 27: warning: exported function NewSlack should have comment or be unexported (golint)
    • Line 34: warning: exported method Slack.Send should have comment or be unexported (golint)
    • Line 102: warning: exported method Slack.Identify should have comment or be unexported (golint)
    • Line 106: warning: comment on exported method Slack.ValidateConfig should be of the form "ValidateConfig ..." (golint)
    • 9volt/monitor/exec_monitor.go
    • Line 14: warning: exported const DEFAULT_CMD_TIMEOUT should have comment (or a comment on this block) or be unexported (golint)
    • Line 17: warning: exported type ExecMonitor should have comment or be unexported (golint)
    • Line 24: warning: exported function NewExecMonitor should have comment or be unexported (golint)
    • Line 45: warning: exported method ExecMonitor.Validate should have comment or be unexported (golint)
    • 9volt/cfgutil/cfgutil.go
    • Line 1: warning: package comment should be of the form "Package cfgutil ..." (golint)
    • Line 19: warning: exported const MONITOR_TYPE should have comment (or a comment on this block) or be unexported (golint)
    • Line 23: warning: exported type CfgUtil should have comment or be unexported (golint)
    • Line 27: warning: exported type YAMLFileBlob should have comment or be unexported (golint)
    • Line 29: warning: comment on exported function New should be of the form "New ..." (golint)
    • Line 41: warning: comment on exported method CfgUtil.Fetch should be of the form "Fetch ..." (golint)
    • Line 79: warning: comment on exported method CfgUtil.Parse should be of the form "Parse ..." (golint)
    • Line 196: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • 9volt/alerter/email_alerter.go
    • Line 15: warning: exported const DEFAULT_EMAIL_FROM should have comment (or a comment on this block) or be unexported (golint)
    • Line 19: warning: exported type Email should have comment or be unexported (golint)
    • Line 24: warning: exported function NewEmail should have comment or be unexported (golint)
    • Line 31: warning: exported method Email.Send should have comment or be unexported (golint)
    • Line 83: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 107: warning: exported method Email.Identify should have comment or be unexported (golint)
    • Line 111: warning: exported method Email.ValidateConfig should have comment or be unexported (golint)
    • 9volt/dal/api_state.go
    • Line 13: warning: comment on exported method Dal.FetchState should be of the form "FetchState ..." (golint)
    • Line 37: warning: comment on exported method Dal.FetchStateWithTags should be of the form "FetchStateWithTags ..." (golint)
    • 9volt/manager/manager.go
    • Line 19: warning: exported type Manager should have comment or be unexported (golint)
    • Line 29: warning: exported function New should have comment or be unexported (golint)
    • Line 42: warning: exported method Manager.Start should have comment or be unexported (golint)
    • Line 52: warning: exported method Manager.Stop should have comment or be unexported (golint)
    • 9volt/event/event.go
    • Line 1: warning: package comment should be of the form "Package event ..." (golint)
    • Line 29: warning: exported const BUFFER_LEN should have comment (or a comment on this block) or be unexported (golint)
    • Line 35: warning: exported type Queue should have comment or be unexported (golint)
    • Line 48: warning: exported type IClient should have comment or be unexported (golint)
    • Line 54: warning: exported type Client should have comment or be unexported (golint)
    • Line 58: warning: exported type Event should have comment or be unexported (golint)
    • Line 65: warning: exported function NewQueue should have comment or be unexported (golint)
    • Line 77: warning: exported method Queue.Start should have comment or be unexported (golint)
    • Line 97: warning: exported method Queue.Stop should have comment or be unexported (golint)
    • Line 139: warning: exported method Queue.NewClient should have comment or be unexported (golint)
    • Line 145: warning: comment on exported method Client.Add should be of the form "Add ..." (golint)
    • Line 163: warning: comment on exported method Client.AddWithErrorLog should be of the form "AddWithErrorLog ..." (golint)
    • Line 168: warning: exported method Client.AddWithLog should have comment or be unexported (golint)
    • 9volt/cluster/cluster.go
    • Line 45: warning: comment on exported const START should be of the form "START ..." (golint)
    • Line 47: warning: exported const STOP should have comment (or a comment on this block) or be unexported (golint)
    • Line 49: warning: comment on exported const CREATE should be of the form "CREATE ..." (golint)
    • Line 57: warning: exported type ICluster should have comment or be unexported (golint)
    • Line 75: warning: exported type Cluster should have comment or be unexported (golint)
    • Line 96: warning: exported type DirectorJSON should have comment or be unexported (golint)
    • Line 101: warning: exported type MemberJSON should have comment or be unexported (golint)
    • Line 118: warning: exported function New should have comment or be unexported (golint)
    • Line 150: warning: exported method Cluster.Start should have comment or be unexported (golint)
    • Line 171: warning: exported method Cluster.Stop should have comment or be unexported (golint)
    • Line 244: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 543: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • 9volt/dal/cfgutil.go
    • Line 13: warning: exported type CfgUtilPushStats should have comment or be unexported (golint)
    • Line 22: warning: comment on exported method Dal.PushConfigs should be of the form "PushConfigs ..." (golint)
    • Line 59: warning: comment on exported method Dal.PushFullConfigs should be of the form "PushFullConfigs ..." (golint)
    • 9volt/api/alerter.go
    • Line 20: warning: comment on exported method Api.AlerterHandler should be of the form "AlerterHandler ..." (golint)
    • Line 60: warning: comment on exported method Api.AlerterAddHandler should be of the form "AlerterAddHandler ..." (golint)
    • Line 101: warning: comment on exported method Api.AlerterDeleteHandler should be of the form "AlerterDeleteHandler ..." (golint)
    • Line 133: warning: exported method Api.AlerterGetHandler should have comment or be unexported (golint)
    • 9volt/api/api.go
    • Line 25: warning: exported type Api should have comment or be unexported (golint)
    • Line 34: warning: exported type JSONStatus should have comment or be unexported (golint)
    • Line 39: warning: exported function New should have comment or be unexported (golint)
    • Line 50: warning: exported method Api.Run should have comment or be unexported (golint)
    • 9volt/api/cluster.go
    • Line 11: warning: comment on exported method Api.ClusterHandler should be of the form "ClusterHandler ..." (golint)
    • 9volt/api/event.go
    • Line 13: warning: comment on exported method Api.EventHandler should be of the form "EventHandler ..." (golint)
    • Line 33: warning: exported method Api.EventWithTypeHandler should have comment or be unexported (golint)
    • 9volt/fakes/basefakes/fake_component.go
    • Line 1: warning: package comment should be of the form "Package basefakes ..." (golint)
    • Line 10: warning: exported type FakeIComponent should have comment or be unexported (golint)
    • Line 42: warning: exported method FakeIComponent.Start should have comment or be unexported (golint)
    • Line 57: warning: exported method FakeIComponent.StartCallCount should have comment or be unexported (golint)
    • Line 63: warning: exported method FakeIComponent.StartReturns should have comment or be unexported (golint)
    • Line 70: warning: exported method FakeIComponent.StartReturnsOnCall should have comment or be unexported (golint)
    • Line 82: warning: exported method FakeIComponent.Stop should have comment or be unexported (golint)
    • Line 97: warning: exported method FakeIComponent.StopCallCount should have comment or be unexported (golint)
    • Line 103: warning: exported method FakeIComponent.StopReturns should have comment or be unexported (golint)
    • Line 110: warning: exported method FakeIComponent.StopReturnsOnCall should have comment or be unexported (golint)
    • Line 122: warning: exported method FakeIComponent.Identify should have comment or be unexported (golint)
    • Line 137: warning: exported method FakeIComponent.IdentifyCallCount should have comment or be unexported (golint)
    • Line 143: warning: exported method FakeIComponent.IdentifyReturns should have comment or be unexported (golint)
    • Line 150: warning: exported method FakeIComponent.IdentifyReturnsOnCall should have comment or be unexported (golint)
    • Line 162: warning: exported method FakeIComponent.Invocations should have comment or be unexported (golint)
    • 9volt/base/base.go
    • Line 9: warning: exported type IComponent should have comment or be unexported (golint)
    • Line 15: warning: exported type Component should have comment or be unexported (golint)
    • Line 22: warning: exported method Component.Identify should have comment or be unexported (golint)
    • 9volt/overwatch/overwatch.go
    • Line 1: warning: package comment should be of the form "Package overwatch ..." (golint)
    • Line 25: warning: exported const WATCH_RETRY_INTERVAL should have comment (or a comment on this block) or be unexported (golint)
    • Line 32: warning: exported type Overwatch should have comment or be unexported (golint)
    • Line 44: warning: exported type Message should have comment or be unexported (golint)
    • Line 50: warning: exported function New should have comment or be unexported (golint)
    • Line 64: warning: exported method Overwatch.Start should have comment or be unexported (golint)
    • Line 73: warning: comment on exported method Overwatch.Stop should be of the form "Stop ..." (golint)
    • 9volt/monitor/dns_monitor.go
    • Line 15: warning: exported const DEFAULT_DNS_TIMEOUT should have comment (or a comment on this block) or be unexported (golint)
    • Line 19: warning: exported type DnsMonitor should have comment or be unexported (golint)
    • Line 27: warning: exported function NewDnsMonitor should have comment or be unexported (golint)
    • Line 62: warning: exported method DnsMonitor.Validate should have comment or be unexported (golint)
    • 9volt/monitor/http_monitor.go
    • Line 15: warning: exported const DEFAULT_HTTP_TIMEOUT should have comment (or a comment on this block) or be unexported (golint)
    • Line 18: warning: exported type HTTPMonitor should have comment or be unexported (golint)
    • Line 24: warning: exported function NewHTTPMonitor should have comment or be unexported (golint)
    • Line 43: warning: exported method HTTPMonitor.Validate should have comment or be unexported (golint)
    • 9volt/monitor/sample_monitor.go
    • Line 15: warning: exported type SampleMonitor should have comment or be unexported (golint)
    • Line 19: warning: exported function NewSampleMonitor should have comment or be unexported (golint)
    • Line 32: warning: exported method SampleMonitor.Validate should have comment or be unexported (golint)
    • 9volt/fakes/etcdclientfakes/fake_keysapi.go
    • Line 1: warning: package comment should be of the form "Package etcdclientfakes ..." (golint)
    • Line 11: warning: exported type FakeKeysAPI should have comment or be unexported (golint)
    • Line 120: warning: exported method FakeKeysAPI.Get should have comment or be unexported (golint)
    • Line 139: warning: exported method FakeKeysAPI.GetCallCount should have comment or be unexported (golint)
    • Line 145: warning: exported method FakeKeysAPI.GetArgsForCall should have comment or be unexported (golint)
    • Line 151: warning: exported method FakeKeysAPI.GetReturns should have comment or be unexported (golint)
    • Line 159: warning: exported method FakeKeysAPI.GetReturnsOnCall should have comment or be unexported (golint)
    • Line 173: warning: exported method FakeKeysAPI.Set should have comment or be unexported (golint)
    • Line 193: warning: exported method FakeKeysAPI.SetCallCount should have comment or be unexported (golint)
    • Line 199: warning: exported method FakeKeysAPI.SetArgsForCall should have comment or be unexported (golint)
    • Line 205: warning: exported method FakeKeysAPI.SetReturns should have comment or be unexported (golint)
    • Line 213: warning: exported method FakeKeysAPI.SetReturnsOnCall should have comment or be unexported (golint)
    • Line 227: warning: exported method FakeKeysAPI.Delete should have comment or be unexported (golint)
    • Line 246: warning: exported method FakeKeysAPI.DeleteCallCount should have comment or be unexported (golint)
    • Line 252: warning: exported method FakeKeysAPI.DeleteArgsForCall should have comment or be unexported (golint)
    • Line 258: warning: exported method FakeKeysAPI.DeleteReturns should have comment or be unexported (golint)
    • Line 266: warning: exported method FakeKeysAPI.DeleteReturnsOnCall should have comment or be unexported (golint)
    • Line 280: warning: exported method FakeKeysAPI.Create should have comment or be unexported (golint)
    • Line 299: warning: exported method FakeKeysAPI.CreateCallCount should have comment or be unexported (golint)
    • Line 305: warning: exported method FakeKeysAPI.CreateArgsForCall should have comment or be unexported (golint)
    • Line 311: warning: exported method FakeKeysAPI.CreateReturns should have comment or be unexported (golint)
    • Line 319: warning: exported method FakeKeysAPI.CreateReturnsOnCall should have comment or be unexported (golint)
    • Line 333: warning: exported method FakeKeysAPI.CreateInOrder should have comment or be unexported (golint)
    • Line 353: warning: exported method FakeKeysAPI.CreateInOrderCallCount should have comment or be unexported (golint)
    • Line 359: warning: exported method FakeKeysAPI.CreateInOrderArgsForCall should have comment or be unexported (golint)
    • Line 365: warning: exported method FakeKeysAPI.CreateInOrderReturns should have comment or be unexported (golint)
    • Line 373: warning: exported method FakeKeysAPI.CreateInOrderReturnsOnCall should have comment or be unexported (golint)
    • Line 387: warning: exported method FakeKeysAPI.Update should have comment or be unexported (golint)
    • Line 406: warning: exported method FakeKeysAPI.UpdateCallCount should have comment or be unexported (golint)
    • Line 412: warning: exported method FakeKeysAPI.UpdateArgsForCall should have comment or be unexported (golint)
    • Line 418: warning: exported method FakeKeysAPI.UpdateReturns should have comment or be unexported (golint)
    • Line 426: warning: exported method FakeKeysAPI.UpdateReturnsOnCall should have comment or be unexported (golint)
    • Line 440: warning: exported method FakeKeysAPI.Watcher should have comment or be unexported (golint)
    • Line 458: warning: exported method FakeKeysAPI.WatcherCallCount should have comment or be unexported (golint)
    • Line 464: warning: exported method FakeKeysAPI.WatcherArgsForCall should have comment or be unexported (golint)
    • Line 470: warning: exported method FakeKeysAPI.WatcherReturns should have comment or be unexported (golint)
    • Line 477: warning: exported method FakeKeysAPI.WatcherReturnsOnCall should have comment or be unexported (golint)
    • Line 489: warning: exported method FakeKeysAPI.Invocations should have comment or be unexported (golint)
    • 9volt/fakes/eventfakes/fake_client.go
    • Line 1: warning: package comment should be of the form "Package eventfakes ..." (golint)
    • Line 11: warning: exported type FakeIClient should have comment or be unexported (golint)
    • Line 55: warning: exported method FakeIClient.Add should have comment or be unexported (golint)
    • Line 73: warning: exported method FakeIClient.AddCallCount should have comment or be unexported (golint)
    • Line 79: warning: exported method FakeIClient.AddArgsForCall should have comment or be unexported (golint)
    • Line 85: warning: exported method FakeIClient.AddReturns should have comment or be unexported (golint)
    • Line 92: warning: exported method FakeIClient.AddReturnsOnCall should have comment or be unexported (golint)
    • Line 104: warning: exported method FakeIClient.AddWithErrorLog should have comment or be unexported (golint)
    • Line 123: warning: exported method FakeIClient.AddWithErrorLogCallCount should have comment or be unexported (golint)
    • Line 129: warning: exported method FakeIClient.AddWithErrorLogArgsForCall should have comment or be unexported (golint)
    • Line 135: warning: exported method FakeIClient.AddWithErrorLogReturns should have comment or be unexported (golint)
    • Line 142: warning: exported method FakeIClient.AddWithErrorLogReturnsOnCall should have comment or be unexported (golint)
    • Line 154: warning: exported method FakeIClient.AddWithLog should have comment or be unexported (golint)
    • Line 174: warning: exported method FakeIClient.AddWithLogCallCount should have comment or be unexported (golint)
    • Line 180: warning: exported method FakeIClient.AddWithLogArgsForCall should have comment or be unexported (golint)
    • Line 186: warning: exported method FakeIClient.AddWithLogReturns should have comment or be unexported (golint)
    • Line 193: warning: exported method FakeIClient.AddWithLogReturnsOnCall should have comment or be unexported (golint)
    • Line 205: warning: exported method FakeIClient.Invocations should have comment or be unexported (golint)
    • 9volt/alerter/pagerduty_alerter.go
    • Line 15: warning: exported const EVENT_TYPE_TRIGGER should have comment (or a comment on this block) or be unexported (golint)
    • Line 19: warning: exported type Pagerduty should have comment or be unexported (golint)
    • Line 24: warning: exported function NewPagerduty should have comment or be unexported (golint)
    • Line 31: warning: exported method Pagerduty.Send should have comment or be unexported (golint)
    • Line 72: warning: exported method Pagerduty.Identify should have comment or be unexported (golint)
    • Line 76: warning: exported method Pagerduty.ValidateConfig 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!


misspell92%

Misspell Finds commonly misspelled English words