gofmt66%

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

• Helen/controllers/controllerhelpers/filters.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/login/twitch.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/config/constants.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/event/event.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/socket/handler/serveme.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/helpers/discord.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/admin/ban.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/login/steam.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/database/db_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/rpc/rpc.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/helpers/amqp.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/helpers/geoip.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/static.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/admin/chatlogs.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/controllerhelpers/slack.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/main.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/websocket.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/admin/lobbies.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/socket/handler/lobby.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/twitch.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/lobby/lobby.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/controllerhelpers/jwt.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/database/db.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/internal/testhelpers/controller.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/controllerhelpers/hooks/player.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/lobby/lobby_requirement.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/admin/dir.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/controllers/admin/stored_servers.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/database/migrations/changes.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/database/migrations/routines.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/helpers/logger.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/models/player/player.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• Helen/helpers/sentry.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

go_vet100%

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

No problems detected. Good job!

gocyclo95%

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.

• Helen/controllers/admin/chatlogs.go
  • Line 32: warning: cyclomatic complexity 16 of function GetChatLogs() is high (> 15) (gocyclo)

• Helen/controllers/socket/handler/lobby.go
  • Line 88: warning: cyclomatic complexity 56 of function (Lobby).LobbyCreate() is high (> 15) (gocyclo)
  • Line 459: warning: cyclomatic complexity 27 of function (Lobby).LobbyJoin() is high (> 15) (gocyclo)

• Helen/models/event/event.go
  • Line 51: warning: cyclomatic complexity 16 of function StartListening() is high (> 15) (gocyclo)

• Helen/models/lobby/lobby.go
  • Line 385: warning: cyclomatic complexity 26 of function (*Lobby).AddPlayer() is high (> 15) (gocyclo)

golint21%

Golint is a linter for Go source code.

• Helen/controllers/controllerhelpers/hooks/player.go
  • Line 22: warning: exported function AfterConnect should have comment or be unexported (golint)
  • Line 32: warning: exported function AfterConnectLoggedIn should have comment or be unexported (golint)

• Helen/database/db.go
  • Line 14: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 27: warning: comment on exported function Init should be of the form "Init ..." (golint)

• Helen/database/migrations/changes.go
  • Line 20: warning: exported type Constant should have comment or be unexported (golint)

• Helen/controllers/admin/dir.go
  • Line 31: warning: exported function ServeAdminPage should have comment or be unexported (golint)

• Helen/models/player/player_decorators.go
  • Line 15: warning: exported method Player.DecoratePlayerTags should have comment or be unexported (golint)
  • Line 65: warning: exported method PlayerBan.MarshalJSON should have comment or be unexported (golint)
  • Line 73: warning: exported method Player.SetPlayerProfile should have comment or be unexported (golint)
  • Line 77: warning: exported method Player.SetPlayerSummary should have comment or be unexported (golint)

• Helen/helpers/recordSync.go
  • Line 20: warning: exported function RLockRecord should have comment or be unexported (golint)
  • Line 33: warning: exported function RUnlockRecord should have comment or be unexported (golint)
  • Line 44: warning: exported function LockRecord should have comment or be unexported (golint)
  • Line 61: warning: exported function UnlockRecord should have comment or be unexported (golint)
  • Line 72: warning: exported function RemoveRecord should have comment or be unexported (golint)

• Helen/internal/version/version.go
  • Line 4: warning: exported var BuildDate should have comment or be unexported (golint)

• Helen/models/gameserver/stored_servers.go
  • Line 10: warning: exported type StoredServer should have comment or be unexported (golint)
  • Line 20: warning: exported var ErrServerUsed should have comment or be unexported (golint)
  • Line 24: warning: exported function NewStoredServer should have comment or be unexported (golint)
  • Line 41: warning: exported function RemoveStoredServer should have comment or be unexported (golint)
  • Line 45: warning: exported function GetAvailableServers should have comment or be unexported (golint)
  • Line 53: warning: exported function GetStoredServer should have comment or be unexported (golint)
  • Line 67: warning: exported function PutStoredServer should have comment or be unexported (golint)
  • Line 73: warning: exported function GetAllStoredServers should have comment or be unexported (golint)

• Helen/models/lobby_settings/lobbySettings.go
  • Line 5: warning: don't use MixedCaps in package name; lobbySettings should be lobbysettings (golint)
  • Line 16: warning: exported type LobbyFormat should have comment or be unexported (golint)
  • Line 23: warning: exported type LobbyMapFormat should have comment or be unexported (golint)
  • Line 29: warning: exported type LobbyMap should have comment or be unexported (golint)
  • Line 35: warning: exported method LobbyMap.GetFormat should have comment or be unexported (golint)
  • Line 50: warning: comment on exported type MapType should be of the form "MapType ..." (with optional leading article) (golint)
  • Line 53: warning: exported type LobbyLeagueDescription should have comment or be unexported (golint)
  • Line 59: warning: exported type LobbyLeagueFormat should have comment or be unexported (golint)
  • Line 65: warning: exported type LobbyLeague should have comment or be unexported (golint)
  • Line 73: warning: exported type LobbyWhitelist should have comment or be unexported (golint)
  • Line 81: warning: exported var LobbyFormats should have comment or be unexported (golint)
  • Line 84: warning: exported var LobbyMaps should have comment or be unexported (golint)
  • Line 87: warning: exported var LobbyLeagues should have comment or be unexported (golint)
  • Line 90: warning: exported var LobbyWhitelists should have comment or be unexported (golint)
  • Line 93: warning: exported function GetLobbyFormat should have comment or be unexported (golint)
  • Line 100: warning: exported function GetLobbyMap should have comment or be unexported (golint)
  • Line 107: warning: exported function GetLobbyLeague should have comment or be unexported (golint)
  • Line 114: warning: exported function GetLobbyWhitelist should have comment or be unexported (golint)
  • Line 121: warning: exported function LoadLobbySettingsFromFile should have comment or be unexported (golint)
  • Line 126: warning: exported function LoadLobbySettings should have comment or be unexported (golint)
  • Line 183: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 216: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 240: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 243: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
  • Line 250: warning: exported function LobbySettingsToJSON should have comment or be unexported (golint)

• Helen/controllers/login/twitch.go
  • Line 27: warning: exported function TwitchLoginHandler should have comment or be unexported (golint)
  • Line 60: warning: exported function TwitchAuthHandler should have comment or be unexported (golint)
  • Line 158: warning: exported function TwitchLogoutHandler should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/claims.go
  • Line 9: warning: exported type TF2StadiumClaims should have comment or be unexported (golint)
  • Line 24: warning: exported method TF2StadiumClaims.Valid should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/jwt.go
  • Line 33: warning: exported function NewToken should have comment or be unexported (golint)
  • Line 60: warning: exported function GetToken should have comment or be unexported (golint)
  • Line 70: warning: exported function GetPlayer should have comment or be unexported (golint)

• Helen/models/lobby/lobby_requirement.go
  • Line 24: warning: exported function NewRequirement should have comment or be unexported (golint)
  • Line 35: warning: exported method Requirement.Save should have comment or be unexported (golint)
  • Line 56: warning: exported method Lobby.HasRequirements should have comment or be unexported (golint)
  • Line 63: warning: receiver name l should be consistent with previous receiver name lobby for Lobby (golint)

• Helen/controllers/socket/server.go
  • Line 11: warning: exported function RegisterHandlers should have comment or be unexported (golint)

• Helen/controllers/socket/handler/global.go
  • Line 17: warning: exported type Global should have comment or be unexported (golint)
  • Line 19: warning: exported method Global.Name should have comment or be unexported (golint)
  • Line 23: warning: exported method Global.GetConstant should have comment or be unexported (golint)
  • Line 47: warning: exported method Global.SendToOtherClients should have comment or be unexported (golint)

• Helen/controllers/socket/handler/mumble.go
  • Line 11: warning: exported method Mumble.Name should have comment or be unexported (golint)
  • Line 15: warning: exported method Mumble.ResetMumblePassword should have comment or be unexported (golint)
  • Line 23: warning: exported method Mumble.GetMumblePassword should have comment or be unexported (golint)

• Helen/models/lobby/lobby.go
  • Line 32: warning: exported type State should have comment or be unexported (golint)
  • Line 35: warning: exported const Initializing should have comment (or a comment on this block) or be unexported (golint)
  • Line 43: warning: exported var ErrLobbyNotFound should have comment or be unexported (golint)
  • Line 56: warning: comment on exported type LobbySlot should be of the form "LobbySlot ..." (with optional leading article) (golint)
  • Line 71: warning: comment on exported function DeleteUnusedServers should be of the form "DeleteUnusedServers ..." (golint)
  • Line 86: warning: exported type TwitchRestriction should have comment or be unexported (golint)
  • Line 89: warning: exported const TwitchSubscribers should have comment (or a comment on this block) or be unexported (golint)
  • Line 180: warning: exported function MapRegionFormatExists should have comment or be unexported (golint)
  • Line 187: warning: comment on exported function NewLobby should be of the form "NewLobby ..." (golint)
  • Line 242: warning: receiver name l should be consistent with previous receiver name lobby for Lobby (golint)
  • Line 248: warning: exported method Lobby.SetState should have comment or be unexported (golint)
  • Line 248: warning: receiver name l should be consistent with previous receiver name lobby for Lobby (golint)
  • Line 255: warning: receiver name l should be consistent with previous receiver name lobby for Lobby (golint)
  • Line 274: warning: exported function RestoreServemeChecks should have comment or be unexported (golint)
  • Line 323: warning: comment on exported function GetLobbyByIDServer should be of the form "GetLobbyByIDServer ..." (golint)
  • Line 628: warning: exported method Lobby.IsPlayerInMumble should have comment or be unexported (golint)
  • Line 703: warning: exported method Lobby.RequiredPlayers should have comment or be unexported (golint)
  • Line 707: warning: exported method Lobby.IsEnoughPlayers should have comment or be unexported (golint)
  • Line 726: warning: comment on exported method Lobby.ShuffleAllSlots should be of the form "ShuffleAllSlots ..." (golint)
  • Line 762: warning: exported method Lobby.DiscordNotif should have comment or be unexported (golint)
  • Line 861: warning: exported method Lobby.DownloadDemo should have comment or be unexported (golint)
  • Line 889: warning: exported method Lobby.UpdateHours should have comment or be unexported (golint)

• Helen/controllers/websocket.go
  • Line 21: warning: exported function SocketHandler should have comment or be unexported (golint)
  • Line 66: warning: exported var ErrRecordNotFound should have comment or be unexported (golint)

• Helen/controllers/login/steam.go
  • Line 26: warning: exported function SteamLoginHandler should have comment or be unexported (golint)
  • Line 47: warning: exported function SteamMockLoginHandler should have comment or be unexported (golint)
  • Line 91: warning: exported function SteamLogoutHandler should have comment or be unexported (golint)
  • Line 107: warning: exported function SteamLoginCallbackHandler should have comment or be unexported (golint)

• Helen/controllers/socket/handler/chat.go
  • Line 22: warning: exported type Chat should have comment or be unexported (golint)
  • Line 24: warning: exported method Chat.Name should have comment or be unexported (golint)
  • Line 28: warning: exported method Chat.ChatSend should have comment or be unexported (golint)
  • Line 75: warning: exported method Chat.ChatDelete should have comment or be unexported (golint)

• Helen/helpers/authorization.go
  • Line 17: warning: exported var RoleNames should have comment or be unexported (golint)
  • Line 24: warning: exported var RoleMap should have comment or be unexported (golint)
  • Line 43: warning: exported var ActionNames should have comment or be unexported (golint)

• Helen/internal/testhelpers/dbhelpers.go
  • Line 14: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 21: warning: exported function CleanupDB should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/hooks/lobby.go
  • Line 23: warning: exported function AfterLobbyJoin should have comment or be unexported (golint)
  • Line 44: warning: exported function AfterLobbyLeave should have comment or be unexported (golint)
  • Line 60: warning: exported function AfterLobbySpec should have comment or be unexported (golint)
  • Line 77: warning: exported function AfterLobbySpecLeave should have comment or be unexported (golint)
  • Line 82: warning: exported function GetLobbyRoom should have comment or be unexported (golint)
  • Line 86: warning: comment on exported function BroadcastLobbyStart should be of the form "BroadcastLobbyStart ..." (golint)

• Helen/database/migrations/routines.go
  • Line 39: warning: don't use underscores in Go names; func whitelist_id_string should be whitelistIDString (golint)

• Helen/models/lobby/format/errors.go
  • Line 7: warning: exported type ErrorInvalidTeam should have comment or be unexported (golint)
  • Line 13: warning: exported type ErrorInvalidClass should have comment or be unexported (golint)
  • Line 19: warning: exported type ErrorInvalidSlot should have comment or be unexported (golint)

• Helen/models/twitch.go
  • Line 16: warning: exported function TFTVStreamStatusUpdater should have comment or be unexported (golint)

• Helen/controllers/mumble.go
  • Line 11: warning: exported function ResetMumblePassword should have comment or be unexported (golint)

• Helen/controllers/template.go
  • Line 9: warning: exported function InitTemplates should have comment or be unexported (golint)

• Helen/models/player/player_report.go
  • Line 9: warning: exported type Report should have comment or be unexported (golint)
  • Line 18: warning: exported type ReportType should have comment or be unexported (golint)
  • Line 21: warning: exported const Substitute should have comment (or a comment on this block) or be unexported (golint)
  • Line 26: warning: exported method Player.NewReport should have comment or be unexported (golint)

• Helen/internal/testhelpers/modelhelpers.go
  • Line 18: warning: exported function CreatePlayer should have comment or be unexported (golint)
  • Line 29: warning: exported function CreatePlayerMod should have comment or be unexported (golint)
  • Line 36: warning: exported function CreatePlayerAdmin should have comment or be unexported (golint)
  • Line 43: warning: exported function CreateLobby should have comment or be unexported (golint)

• Helen/controllers/socket/handler/unauth.go
  • Line 15: warning: exported type Unauth should have comment or be unexported (golint)
  • Line 17: warning: exported method Unauth.Name should have comment or be unexported (golint)
  • Line 21: warning: exported method Unauth.LobbySpectatorJoin should have comment or be unexported (golint)
  • Line 38: warning: exported method Unauth.LobbySpectatorLeave should have comment or be unexported (golint)
  • Line 51: warning: exported method Unauth.PlayerProfile should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/ip.go
  • Line 10: warning: exported function GetIPAddr should have comment or be unexported (golint)

• Helen/database/migrations/migrations.go
  • Line 20: warning: exported function Do should have comment or be unexported (golint)

• Helen/models/chat/chat.go
  • Line 29: warning: comment on exported function NewChatMessage should be of the form "NewChatMessage ..." (golint)
  • Line 42: warning: exported function NewInGameChatMessage should have comment or be unexported (golint)
  • Line 52: warning: exported method ChatMessage.Save should have comment or be unexported (golint)
  • Line 54: warning: exported method ChatMessage.Send should have comment or be unexported (golint)
  • Line 70: warning: exported method ChatMessage.MarshalJSON should have comment or be unexported (golint)
  • Line 105: warning: exported function NewBotMessage should have comment or be unexported (golint)
  • Line 117: warning: exported function SendNotification should have comment or be unexported (golint)
  • Line 122: warning: comment on exported function GetRoomMessages should be of the form "GetRoomMessages ..." (golint)
  • Line 131: warning: comment on exported function GetPlayerMessages should be of the form "GetPlayerMessages ..." (golint)
  • Line 141: warning: comment on exported function GetScrollback should be of the form "GetScrollback ..." (golint)

• Helen/models/event/event.go
  • Line 19: warning: comment on exported type Event should be of the form "Event ..." (with optional leading article) (golint)
  • Line 51: warning: exported function StartListening should have comment or be unexported (golint)
  • Line 99: warning: exported function StopListening should have comment or be unexported (golint)

• Helen/models/rpc/pauling.go
  • Line 12: warning: exported type Args should have comment or be unexported (golint)
  • Line 26: warning: exported function DisallowPlayer should have comment or be unexported (golint)
  • Line 38: warning: exported function SetupServer should have comment or be unexported (golint)
  • Line 54: warning: exported function ReExecConfig should have comment or be unexported (golint)
  • Line 61: warning: exported function VerifyInfo should have comment or be unexported (golint)
  • Line 68: warning: exported function End should have comment or be unexported (golint)
  • Line 75: warning: exported function Say should have comment or be unexported (golint)

• Helen/controllers/twitch_badge.go
  • Line 24: warning: exported function TwitchBadge should have comment or be unexported (golint)

• Helen/models/player/player_stats.go
  • Line 14: warning: exported type PlayerStats should have comment or be unexported (golint)
  • Line 47: warning: exported function NewStats should have comment or be unexported (golint)
  • Line 53: warning: exported method PlayerStats.Save should have comment or be unexported (golint)
  • Line 57: warning: exported method PlayerStats.TotalLobbies should have comment or be unexported (golint)
  • Line 61: warning: exported method PlayerStats.PlayedCountIncrease should have comment or be unexported (golint)
  • Line 79: warning: exported method PlayerStats.IncreaseSubCount should have comment or be unexported (golint)
  • Line 84: warning: exported method PlayerStats.IncreaseClassCount should have comment or be unexported (golint)

• Helen/controllers/broadcaster/broadcaster.go
  • Line 14: warning: exported function SendMessage should have comment or be unexported (golint)
  • Line 29: warning: exported function SendMessageToRoom should have comment or be unexported (golint)
  • Line 36: warning: exported function SendMessageSkipIDs should have comment or be unexported (golint)

• Helen/helpers/authority/authority.go
  • Line 9: warning: exported type AuthAction should have comment or be unexported (golint)
  • Line 11: warning: exported type AuthRole should have comment or be unexported (golint)
  • Line 20: warning: exported method AuthRole.Allow should have comment or be unexported (golint)
  • Line 31: warning: exported method AuthRole.Disallow should have comment or be unexported (golint)
  • Line 42: warning: exported method AuthRole.Inherit should have comment or be unexported (golint)
  • Line 42: warning: receiver name myrole should be consistent with previous receiver name role for AuthRole (golint)
  • Line 61: warning: exported method AuthRole.Can should have comment or be unexported (golint)
  • Line 66: warning: exported function Can should have comment or be unexported (golint)
  • Line 66: warning: don't use underscores in Go names; func parameter role_int should be roleInt (golint)
  • Line 71: warning: exported function Reset should have comment or be unexported (golint)

• Helen/models/rpc/twitchbot.go
  • Line 3: warning: exported function TwitchBotJoin should have comment or be unexported (golint)
  • Line 10: warning: exported function TwitchBotLeave should have comment or be unexported (golint)
  • Line 17: warning: exported function TwitchBotAnnouce should have comment or be unexported (golint)

• Helen/controllers/socket/handler/lobby.go
  • Line 35: warning: exported type Lobby should have comment or be unexported (golint)
  • Line 37: warning: exported method Lobby.Name should have comment or be unexported (golint)
  • Line 56: warning: exported type Restriction should have comment or be unexported (golint)
  • Line 60: warning: exported type Requirement should have comment or be unexported (golint)
  • Line 88: warning: exported method Lobby.LobbyCreate should have comment or be unexported (golint)
  • Line 360: warning: exported method Lobby.LobbyServerReset should have comment or be unexported (golint)
  • Line 388: warning: exported method Lobby.ServerVerify should have comment or be unexported (golint)
  • Line 418: warning: exported method Lobby.LobbyClose should have comment or be unexported (golint)
  • Line 459: warning: exported method Lobby.LobbyJoin should have comment or be unexported (golint)
  • Line 591: warning: exported method Lobby.LobbySpectatorJoin should have comment or be unexported (golint)
  • Line 680: warning: exported method Lobby.LobbyKick should have comment or be unexported (golint)
  • Line 708: warning: exported method Lobby.LobbyBan should have comment or be unexported (golint)
  • Line 738: warning: exported method Lobby.LobbyLeave should have comment or be unexported (golint)
  • Line 754: warning: exported method Lobby.LobbySpectatorLeave should have comment or be unexported (golint)
  • Line 777: warning: exported method Lobby.RequestLobbyListData should have comment or be unexported (golint)
  • Line 783: warning: exported method Lobby.LobbyChangeOwner should have comment or be unexported (golint)
  • Line 813: warning: exported method Lobby.LobbySetRequirement should have comment or be unexported (golint)
  • Line 882: warning: exported method Lobby.LobbySetTeamName should have comment or be unexported (golint)
  • Line 915: warning: exported method Lobby.LobbyRemoveTwitchRestriction should have comment or be unexported (golint)
  • Line 938: warning: exported method Lobby.LobbyRemoveSteamRestriction should have comment or be unexported (golint)
  • Line 961: warning: exported method Lobby.LobbyRemoveRegionLock should have comment or be unexported (golint)
  • Line 985: warning: exported method Lobby.LobbyShuffle should have comment or be unexported (golint)

• Helen/helpers/whitelist.go
  • Line 10: warning: exported type GroupXML should have comment or be unexported (golint)
  • Line 16: warning: comment on exported function GetGroupMembers should be of the form "GetGroupMembers ..." (golint)
  • Line 39: warning: exported function IsWhitelisted should have comment or be unexported (golint)

• Helen/models/gameserver/server.go
  • Line 3: warning: exported type ServerRecord should have comment or be unexported (golint)

• Helen/controllers/socket/handler/serveme.go
  • Line 15: warning: exported type Serveme should have comment or be unexported (golint)
  • Line 17: warning: exported method Serveme.Name should have comment or be unexported (golint)
  • Line 21: warning: exported method Serveme.GetServemeServers should have comment or be unexported (golint)
  • Line 55: warning: exported method Serveme.GetStoredServers should have comment or be unexported (golint)

• Helen/models/player/player_bans.go
  • Line 10: warning: exported type BanType should have comment or be unexported (golint)
  • Line 13: warning: exported const BanJoin should have comment (or a comment on this block) or be unexported (golint)
  • Line 45: warning: exported method Player.IsBannedWithTime should have comment or be unexported (golint)
  • Line 55: warning: exported method Player.IsBanned should have comment or be unexported (golint)
  • Line 60: warning: exported method Player.BanUntil should have comment or be unexported (golint)
  • Line 77: warning: exported method Player.Unban should have comment or be unexported (golint)
  • Line 82: warning: exported method Player.GetActiveBan should have comment or be unexported (golint)
  • Line 95: warning: exported method Player.GetActiveBans should have comment or be unexported (golint)
  • Line 101: warning: exported method Player.GetAllBans should have comment or be unexported (golint)
  • Line 108: warning: exported function GetAllActiveBans should have comment or be unexported (golint)
  • Line 114: warning: exported function GetAllBans should have comment or be unexported (golint)

• Helen/models/rpc/fumble.go
  • Line 5: warning: exported function FumbleLobbyCreated should have comment or be unexported (golint)
  • Line 20: warning: exported function FumbleLobbyEnded should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/slack.go
  • Line 21: warning: exported type SlackMessage should have comment or be unexported (golint)
  • Line 44: warning: exported function SendToSlack should have comment or be unexported (golint)

• Helen/models/player/player.go
  • Line 31: warning: exported var ErrPlayerNotFound should have comment or be unexported (golint)
  • Line 32: warning: exported var ErrPlayerInReportedSlot should have comment or be unexported (golint)
  • Line 66: warning: comment on exported type JSONFields should be of the form "JSONFields ..." (with optional leading article) (golint)
  • Line 79: warning: comment on exported function NewPlayer should be of the form "NewPlayer ..." (golint)
  • Line 102: warning: exported method Player.SetExternalLinks should have comment or be unexported (golint)
  • Line 162: warning: exported method Player.GenAuthKey should have comment or be unexported (golint)
  • Line 200: warning: exported method Player.SetMumbleUsername should have comment or be unexported (golint)
  • Line 227: warning: comment on exported method Player.Alias should be of the form "Alias ..." (golint)
  • Line 228: warning: receiver name p should be consistent with previous receiver name player for Player (golint)
  • Line 248: warning: comment on exported function GetPlayerByID should be of the form "GetPlayerByID ..." (golint)
  • Line 259: warning: comment on exported function GetPlayerBySteamID should be of the form "GetPlayerBySteamID ..." (golint)
  • Line 269: warning: comment on exported function GetPlayerWithStats should be of the form "GetPlayerWithStats ..." (golint)
  • Line 279: warning: comment on exported method Player.GetLobbyID should be of the form "GetLobbyID ..." (golint)
  • Line 300: warning: comment on exported method Player.IsSpectatingID should be of the form "IsSpectatingID ..." (golint)
  • Line 307: warning: comment on exported method Player.GetSpectatingIds should be of the form "GetSpectatingIds ..." (golint)
  • Line 322: warning: comment on exported method Player.UpdatePlayerInfo should be of the form "UpdatePlayerInfo ..." (golint)
  • Line 362: warning: exported method Player.SetSetting should have comment or be unexported (golint)
  • Line 371: warning: exported method Player.GetSetting should have comment or be unexported (golint)
  • Line 386: warning: receiver name p should be consistent with previous receiver name player for Player (golint)
  • Line 411: warning: comment on exported method Player.HasSubscriptionProgram should be of the form "HasSubscriptionProgram ..." (golint)
  • Line 413: warning: receiver name p should be consistent with previous receiver name player for Player (golint)
  • Line 432: warning: receiver name p should be consistent with previous receiver name player for Player (golint)
  • Line 451: warning: exported method Player.HasCreatedLobby should have comment or be unexported (golint)
  • Line 451: warning: receiver name p should be consistent with previous receiver name player for Player (golint)
  • Line 461: warning: receiver name p should be consistent with previous receiver name player for Player (golint)

• Helen/helpers/syslog_unix.go
  • Line 3: warning: package comment should be of the form "Package helpers ..." (golint)

• Helen/models/lobby/format/formats.go
  • Line 3: warning: exported type Format should have comment or be unexported (golint)
  • Line 6: warning: exported const Sixes should have comment (or a comment on this block) or be unexported (golint)
  • Line 108: warning: exported var NumberOfClassesMap should have comment or be unexported (golint)
  • Line 170: warning: exported function GetClasses should have comment or be unexported (golint)

• Helen/models/rpc/rpc.go
  • Line 27: warning: exported function ConnectRPC should have comment or be unexported (golint)

• Helen/controllers/socket/handler/player.go
  • Line 20: warning: exported type Player should have comment or be unexported (golint)
  • Line 22: warning: exported method Player.Name should have comment or be unexported (golint)
  • Line 26: warning: exported method Player.PlayerReady should have comment or be unexported (golint)
  • Line 58: warning: exported method Player.PlayerNotReady should have comment or be unexported (golint)
  • Line 92: warning: exported method Player.PlayerSettingsGet should have comment or be unexported (golint)
  • Line 107: warning: exported method Player.PlayerSettingsSet should have comment or be unexported (golint)
  • Line 138: warning: exported method Player.PlayerProfile should have comment or be unexported (golint)
  • Line 163: warning: exported method Player.PlayerEnableTwitchBot should have comment or be unexported (golint)
  • Line 191: warning: exported method Player.PlayerDisableTwitchBot should have comment or be unexported (golint)
  • Line 218: warning: exported method Player.PlayerRecentLobbies should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/chatScrollback.go
  • Line 9: warning: exported function BroadcastScrollback should have comment or be unexported (golint)

• Helen/controllers/controllerhelpers/filters.go
  • Line 26: warning: exported function WhitelistListener should have comment or be unexported (golint)
  • Line 63: warning: exported function IsSteamIDWhitelisted should have comment or be unexported (golint)
  • Line 71: warning: comment on exported function CheckPrivilege should be of the form "CheckPrivilege ..." (golint)
  • Line 81: warning: exported function FilterHTTPRequest should have comment or be unexported (golint)

• Helen/helpers/geoip.go
  • Line 15: warning: exported function InitGeoIPDB should have comment or be unexported (golint)
  • Line 28: warning: exported function GetRegion should have comment or be unexported (golint)

• Helen/helpers/serveme.go
  • Line 12: warning: exported var ServemeNA should have comment or be unexported (golint)
  • Line 17: warning: exported function GetServemeContextIP should have comment or be unexported (golint)
  • Line 38: warning: exported function GetServemeContext should have comment or be unexported (golint)

• Helen/internal/testhelpers/controller.go
  • Line 26: warning: exported const InitMessages should have comment or be unexported (golint)
  • Line 28: warning: exported type SuffixList should have comment or be unexported (golint)
  • Line 34: warning: exported method SuffixList.PublicSuffix should have comment or be unexported (golint)
  • Line 42: warning: exported var DefaultTransport should have comment or be unexported (golint)
  • Line 50: warning: exported function NewClient should have comment or be unexported (golint)
  • Line 58: warning: exported function Login should have comment or be unexported (golint)
  • Line 63: warning: exported function ConnectWS should have comment or be unexported (golint)
  • Line 77: warning: exported function LoginAndConnectWS should have comment or be unexported (golint)
  • Line 96: warning: exported function EmitJSONWithReply should have comment or be unexported (golint)
  • Line 110: warning: exported function StartServer should have comment or be unexported (golint)
  • Line 127: warning: exported function ReadMessages should have comment or be unexported (golint)
  • Line 142: warning: exported function ReadJSON should have comment or be unexported (golint)

• Helen/controllers/admin/ban.go
  • Line 22: warning: exported function BanPlayer should have comment or be unexported (golint)
  • Line 88: warning: exported function GetBanLogs should have comment or be unexported (golint)

• Helen/internal/testhelpers/socket.go
  • Line 7: warning: comment on exported function SocketCreateLobby should be of the form "SocketCreateLobby ..." (golint)
  • Line 26: warning: comment on exported function SocketJoinLobby should be of the form "SocketJoinLobby ..." (golint)

• Helen/controllers/admin/stored_servers.go
  • Line 16: warning: exported function AddServer should have comment or be unexported (golint)
  • Line 54: warning: exported function RemoveServer should have comment or be unexported (golint)
  • Line 74: warning: exported function ViewServerPage should have comment or be unexported (golint)

• Helen/controllers/admin/templates.go
  • Line 7: warning: exported function InitAdminTemplates should have comment or be unexported (golint)

• Helen/routes/socket/middleware/codec.go
  • Line 10: warning: exported type JSONCodec should have comment or be unexported (golint)
  • Line 12: warning: exported method JSONCodec.ReadName should have comment or be unexported (golint)
  • Line 20: warning: exported method JSONCodec.Unmarshal should have comment or be unexported (golint)

• Helen/models/admin_log.go
  • Line 14: warning: exported type AdminLogEntry should have comment or be unexported (golint)
  • Line 21: warning: exported function LogCustomAdminAction should have comment or be unexported (golint)
  • Line 31: warning: exported function LogAdminAction should have comment or be unexported (golint)

• Helen/controllers/static.go
  • Line 24: warning: exported function MainHandler should have comment or be unexported (golint)

• Helen/helpers/amqp.go
  • Line 9: warning: exported var AMQPChannel should have comment or be unexported (golint)
  • Line 10: warning: exported var AMQPConn should have comment or be unexported (golint)
  • Line 12: warning: exported function ConnectAMQP should have comment or be unexported (golint)

• Helen/helpers/discord.go
  • Line 16: warning: exported var Discord should have comment or be unexported (golint)
  • Line 21: warning: exported function DiscordSendToChannel should have comment or be unexported (golint)
  • Line 31: warning: exported function DiscordEmoji should have comment or be unexported (golint)

• Helen/models/lobby/lobby_decorators.go
  • Line 18: warning: exported type SlotDetails should have comment or be unexported (golint)
  • Line 29: warning: exported type ClassDetails should have comment or be unexported (golint)
  • Line 35: warning: exported type SpecDetails should have comment or be unexported (golint)
  • Line 40: warning: exported type LobbyData should have comment or be unexported (golint)
  • Line 74: warning: exported type LobbyListData should have comment or be unexported (golint)
  • Line 78: warning: exported type LobbyConnectData should have comment or be unexported (golint)
  • Line 97: warning: exported type SubstituteData should have comment or be unexported (golint)
  • Line 118: warning: exported type LobbyEvent should have comment or be unexported (golint)
  • Line 163: warning: exported function DecorateLobbyData should have comment or be unexported (golint)
  • Line 237: warning: exported method LobbyData.Send should have comment or be unexported (golint)
  • Line 241: warning: exported method LobbyData.SendToPlayer should have comment or be unexported (golint)
  • Line 245: warning: exported function DecorateLobbyListData should have comment or be unexported (golint)
  • Line 256: warning: exported function DecorateLobbyConnect should have comment or be unexported (golint)
  • Line 279: warning: exported function DecorateLobbyJoin should have comment or be unexported (golint)
  • Line 283: warning: exported function DecorateLobbyClosed should have comment or be unexported (golint)
  • Line 287: warning: exported function DecorateSubstitute should have comment or be unexported (golint)
  • Line 312: warning: exported function DecorateSubstituteList should have comment or be unexported (golint)

• Helen/config/constants.go
  • Line 68: warning: exported var Constants should have comment or be unexported (golint)
  • Line 100: warning: exported function PrintConfigDoc should have comment or be unexported (golint)

• Helen/controllers/stats/stats.go
  • Line 11: warning: exported var Stats should have comment or be unexported (golint)
  • Line 22: warning: exported function StatsHandler should have comment or be unexported (golint)

• Helen/controllers/admin/lobbies.go
  • Line 15: warning: exported function ViewOpenLobbies should have comment or be unexported (golint)

• Helen/controllers/admin/chatlogs.go
  • Line 32: warning: exported function GetChatLogs should have comment or be unexported (golint)

• Helen/controllers/admin/roles.go
  • Line 18: warning: exported function ChangeRole should have comment or be unexported (golint)
  • Line 62: warning: exported function Remove should have comment or be unexported (golint)

• Helen/routes/http.go
  • Line 50: warning: exported function SetupHTTP should have comment or be unexported (golint)

• Helen/helpers/httpclient.go
  • Line 8: warning: exported var HTTPClient should have comment or be unexported (golint)

• Helen/helpers/request.go
  • Line 3: warning: exported type Request should have comment or be unexported (golint)
  • Line 8: warning: exported function NewRequest should have comment or be unexported (golint)

• Helen/helpers/sentry.go
  • Line 13: warning: exported var Raven should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign94%

IneffAssign detects ineffectual assignments in Go code.

• Helen/controllers/twitch_badge.go
  • Line 47: warning: ineffectual assignment to err (ineffassign)

• Helen/controllers/socket/handler/lobby.go
  • Line 604: warning: ineffectual assignment to tperr (ineffassign)

• Helen/controllers/socket/handler/player.go
  • Line 74: warning: ineffectual assignment to err (ineffassign)

• Helen/models/lobby/format/formats_test.go
  • Line 35: warning: ineffectual assignment to res (ineffassign)
  • Line 38: warning: ineffectual assignment to res (ineffassign)

• Helen/models/lobby/lobby_test.go
  • Line 128: warning: ineffectual assignment to id (ineffassign)
  • Line 338: warning: ineffectual assignment to err (ineffassign)

misspell96%

Misspell Finds commonly misspelled English words

• Helen/controllers/admin/chatlogs.go
  • Line 81: warning: "specfic" is a misspelling of "specific" (misspell)

• Helen/controllers/controllerhelpers/filters.go
  • Line 73: warning: "neccesary" is a misspelling of "necessary" (misspell)

• Helen/models/lobby/lobby.go
  • Line 72: warning: "corresponsing" is a misspelling of "corresponding" (misspell)
  • Line 874: warning: "relevent" is a misspelling of "relevant" (misspell)