Report for github.com/virtual-vgo/vvgo
A+ Excellent! Found 40 issues across 64 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
golint37%
Golint is a linter for Go source code.
-
-
vvgo/pkg/http_wrappers/wrappers.go
- Line 1: warning: don't use an underscore in package name (golint)
- Line 36: warning: exported function Handler should have comment or be unexported (golint)
- Line 59: warning: exported function DoRequest should have comment or be unexported (golint)
- Line 84: warning: exported function Get should have comment or be unexported (golint)
- Line 92: warning: exported function PostForm should have comment or be unexported (golint)
- Line 101: warning: exported var DebugHTTP should have comment or be unexported (golint)
-
-
vvgo/pkg/api/intent_stats.go
- Line 14: warning: exported const SkywardSwordIntentMessageID should have comment or be unexported (golint)
- Line 15: warning: exported const SkywardSwordStatsChannelID should have comment or be unexported (golint)
- Line 17: warning: exported function SkywardSwordIntentHandler should have comment or be unexported (golint)
-
-
vvgo/cmd/vvgo/main.go
- Line 17: warning: exported type Flags should have comment or be unexported (golint)
- Line 22: warning: exported method Flags.Parse should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/projects.go
- Line 9: warning: exported type Project should have comment or be unexported (golint)
- Line 39: warning: exported method Project.ProjectPage should have comment or be unexported (golint)
- Line 40: warning: exported method Project.PartsPage should have comment or be unexported (golint)
- Line 42: warning: exported type Projects should have comment or be unexported (golint)
- Line 44: warning: exported function ListProjects should have comment or be unexported (golint)
- Line 65: warning: exported method Projects.Current should have comment or be unexported (golint)
- Line 71: warning: exported method Projects.Get should have comment or be unexported (golint)
- Line 80: warning: exported method Projects.Has should have comment or be unexported (golint)
- Line 82: warning: exported method Projects.ForIdentity should have comment or be unexported (golint)
- Line 97: warning: exported method Projects.Select should have comment or be unexported (golint)
- Line 108: warning: exported method Projects.Append should have comment or be unexported (golint)
- Line 110: warning: exported method Projects.First should have comment or be unexported (golint)
- Line 113: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
- Line 118: warning: exported method Projects.Last should have comment or be unexported (golint)
- Line 121: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
- Line 126: warning: exported method Projects.WithField should have comment or be unexported (golint)
- Line 130: warning: exported method Projects.WithFields should have comment or be unexported (golint)
- Line 134: warning: exported method Projects.Query should have comment or be unexported (golint)
- Line 140: warning: exported method Projects.Names should have comment or be unexported (golint)
- Line 153: warning: exported method Projects.Sort should have comment or be unexported (golint)
- Line 154: warning: exported method Projects.ReverseSort should have comment or be unexported (golint)
-
-
vvgo/pkg/api/leaders.go
- Line 8: warning: exported function LeadersApi should have comment or be unexported (golint)
-
-
vvgo/pkg/login/access.go
- Line 9: warning: comment on exported type Kind should be of the form "Kind ..." (with optional leading article) (golint)
- Line 16: warning: exported const KindPassword should have comment (or a comment on this block) or be unexported (golint)
- Line 22: warning: comment on exported type Role should be of the form "Role ..." (with optional leading article) (golint)
- Line 30: warning: exported const RoleAnonymous should have comment (or a comment on this block) or be unexported (golint)
- Line 41: warning: exported function Anonymous should have comment or be unexported (golint)
- Line 43: warning: comment on exported type Identity should be of the form "Identity ..." (with optional leading article) (golint)
- Line 52: warning: exported method Identity.Info should have comment or be unexported (golint)
- Line 60: warning: exported method Identity.HasRole should have comment or be unexported (golint)
- Line 72: warning: exported method Identity.IsAnonymous should have comment or be unexported (golint)
- Line 76: warning: exported method Identity.AssumeRoles should have comment or be unexported (golint)
-
-
vvgo/pkg/api/login_test.go
- Line 34: warning: should not use basic type string as key in context.WithValue (golint)
-
-
vvgo/pkg/api/routes.go
- Line 12: warning: exported var PublicFiles should have comment or be unexported (golint)
- Line 14: warning: exported function Routes should have comment or be unexported (golint)
-
-
vvgo/pkg/api/slash_commands.go
- Line 20: warning: exported var SlashCommands should have comment or be unexported (golint)
- Line 51: warning: exported function CreateSlashCommands should have comment or be unexported (golint)
- Line 61: warning: exported function ViewSlashCommands should have comment or be unexported (golint)
- Line 72: warning: exported function HandleSlashCommand should have comment or be unexported (golint)
- Line 116: warning: exported function HandleInteraction should have comment or be unexported (golint)
- Line 135: warning: exported type SlashCommand should have comment or be unexported (golint)
- Line 142: warning: exported type InteractionHandler should have comment or be unexported (golint)
- Line 144: warning: exported method SlashCommand.Create should have comment or be unexported (golint)
- Line 271: warning: exported var InteractionResponseOof should have comment or be unexported (golint)
-
-
vvgo/pkg/api/voting.go
- Line 15: warning: exported var VotingView should have comment or be unexported (golint)
- Line 17: warning: exported var ArrangementsBallotApi should have comment or be unexported (golint)
- Line 84: warning: exported var VotingResultsView should have comment or be unexported (golint)
-
-
vvgo/pkg/when2meet/when2meet.go
- Line 15: warning: exported const TimeZone should have comment or be unexported (golint)
- Line 17: warning: exported var Endpoint should have comment or be unexported (golint)
- Line 40: warning: exported function CreateEvent should have comment or be unexported (golint)
-
-
vvgo/pkg/http_wrappers/wrappers_test.go
- Line 1: warning: don't use an underscore in package name (golint)
-
-
vvgo/pkg/foaas/foaas.go
- Line 13: warning: exported const Endpoint should have comment or be unexported (golint)
- Line 14: warning: exported const OperationsEndpoint should have comment or be unexported (golint)
- Line 16: warning: exported type Operation should have comment or be unexported (golint)
- Line 25: warning: exported type Response should have comment or be unexported (golint)
- Line 30: warning: exported function FuckOff should have comment or be unexported (golint)
-
-
vvgo/pkg/api/auth.go
- Line 11: warning: exported const CtxKeyVVGOIdentity should have comment or be unexported (golint)
- Line 13: warning: exported function IdentityFromContext should have comment or be unexported (golint)
- Line 23: warning: comment on exported type RBACMux should be of the form "RBACMux ..." (with optional leading article) (golint)
- Line 36: warning: exported method RBACMux.Handle should have comment or be unexported (golint)
- Line 63: warning: should not use basic type string as key in context.WithValue (golint)
-
-
vvgo/pkg/api/credits_maker.go
- Line 9: warning: exported type CreditsMakerConfig should have comment or be unexported (golint)
- Line 15: warning: exported var CreditsMaker should have comment or be unexported (golint)
-
-
vvgo/pkg/api/login.go
- Line 18: warning: exported const LoginCookieDuration should have comment or be unexported (golint)
- Line 20: warning: exported type LoginView should have comment or be unexported (golint)
- Line 22: warning: exported const CookieLoginRedirect should have comment or be unexported (golint)
- Line 51: warning: exported type LoginSuccessView should have comment or be unexported (golint)
- Line 57: warning: exported type LoginRedirect should have comment or be unexported (golint)
- Line 75: warning: context.Context should be the first parameter of a function (golint)
- Line 134: warning: comment on exported type DiscordLoginHandler should be of the form "DiscordLoginHandler ..." (with optional leading article) (golint)
- Line 140: warning: exported var ErrNotAMember should have comment or be unexported (golint)
- Line 215: warning: exported const CookieOAuthState should have comment or be unexported (golint)
- Line 242: warning: context.Context should be the first parameter of a function (golint)
-
-
vvgo/pkg/api/template.go
- Line 16: warning: exported function ParseAndExecute should have comment or be unexported (golint)
- Line 63: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
-
-
vvgo/pkg/minio/minio.go
- Line 16: warning: exported type Config should have comment or be unexported (golint)
- Line 33: warning: exported type Client should have comment or be unexported (golint)
- Line 35: warning: exported function NewClient should have comment or be unexported (golint)
- Line 46: warning: exported method Client.NewRandomBucket should have comment or be unexported (golint)
-
-
vvgo/pkg/version/version.go
- Line 13: warning: exported const VersionFile should have comment or be unexported (golint)
- Line 25: warning: exported function Set should have comment or be unexported (golint)
- Line 26: warning: exported function String should have comment or be unexported (golint)
- Line 27: warning: exported function JSON should have comment or be unexported (golint)
- Line 28: warning: exported function Header should have comment or be unexported (golint)
- Line 29: warning: exported function SetVersionHeaders should have comment or be unexported (golint)
- Line 36: warning: exported type Version should have comment or be unexported (golint)
- Line 48: warning: exported method Version.JSON should have comment or be unexported (golint)
- Line 53: warning: exported method Version.Header should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/parts.go
- Line 8: warning: exported type Part should have comment or be unexported (golint)
- Line 23: warning: exported type Parts should have comment or be unexported (golint)
- Line 25: warning: exported function ListParts should have comment or be unexported (golint)
- Line 51: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
- Line 56: warning: exported method Parts.ForProject should have comment or be unexported (golint)
- Line 68: warning: exported method Parts.Append should have comment or be unexported (golint)
- Line 77: warning: exported method Parts.Sort should have comment or be unexported (golint)
-
-
vvgo/pkg/discord/models.go
- Line 3: warning: comment on exported type OAuthToken should be of the form "OAuthToken ..." (with optional leading article) (golint)
- Line 12: warning: comment on exported type Snowflake should be of the form "Snowflake ..." (with optional leading article) (golint)
- Line 17: warning: comment on exported type User should be of the form "User ..." (with optional leading article) (golint)
- Line 22: warning: comment on exported type GuildMember should be of the form "GuildMember ..." (with optional leading article) (golint)
- Line 29: warning: comment on exported type CreateMessageParams should be of the form "CreateMessageParams ..." (with optional leading article) (golint)
- Line 35: warning: exported type EditMessageParams should have comment or be unexported (golint)
- Line 37: warning: comment on exported type CreateApplicationCommandParams should be of the form "CreateApplicationCommandParams ..." (with optional leading article) (golint)
- Line 44: warning: comment on exported type ApplicationCommand should be of the form "ApplicationCommand ..." (with optional leading article) (golint)
- Line 53: warning: comment on exported type ApplicationCommandOption should be of the form "ApplicationCommandOption ..." (with optional leading article) (golint)
- Line 63: warning: comment on exported type ApplicationCommandOptionType should be of the form "ApplicationCommandOptionType ..." (with optional leading article) (golint)
- Line 67: warning: exported const ApplicationCommandOptionTypeSubCommand should have comment (or a comment on this block) or be unexported (golint)
- Line 77: warning: comment on exported type ApplicationCommandOptionChoice should be of the form "ApplicationCommandOptionChoice ..." (with optional leading article) (golint)
- Line 83: warning: comment on exported type Interaction should be of the form "Interaction ..." (with optional leading article) (golint)
- Line 94: warning: comment on exported type InteractionType should be of the form "InteractionType ..." (with optional leading article) (golint)
- Line 98: warning: exported const InteractionTypePing should have comment (or a comment on this block) or be unexported (golint)
- Line 102: warning: comment on exported type ApplicationCommandInteractionData should be of the form "ApplicationCommandInteractionData ..." (with optional leading article) (golint)
- Line 109: warning: comment on exported type ApplicationCommandInteractionDataOption should be of the form "ApplicationCommandInteractionDataOption ..." (with optional leading article) (golint)
- Line 116: warning: comment on exported type InteractionResponse should be of the form "InteractionResponse ..." (with optional leading article) (golint)
- Line 122: warning: comment on exported type InteractionResponseType should be of the form "InteractionResponseType ..." (with optional leading article) (golint)
- Line 126: warning: exported const InteractionResponseTypePong should have comment (or a comment on this block) or be unexported (golint)
- Line 133: warning: comment on exported type InteractionApplicationCommandCallbackData should be of the form "InteractionApplicationCommandCallbackData ..." (with optional leading article) (golint)
- Line 140: warning: comment on exported type ExecuteWebhookParams should be of the form "ExecuteWebhookParams ..." (with optional leading article) (golint)
- Line 149: warning: comment on exported type Embed should be of the form "Embed ..." (with optional leading article) (golint)
- Line 165: warning: comment on exported type EmbedType should be of the form "EmbedType ..." (with optional leading article) (golint)
- Line 169: warning: exported const EmbedTypeRich should have comment (or a comment on this block) or be unexported (golint)
- Line 177: warning: comment on exported type EmbedThumbnail should be of the form "EmbedThumbnail ..." (with optional leading article) (golint)
- Line 185: warning: comment on exported type EmbedVideo should be of the form "EmbedVideo ..." (with optional leading article) (golint)
- Line 193: warning: comment on exported type EmbedImage should be of the form "EmbedImage ..." (with optional leading article) (golint)
- Line 201: warning: comment on exported type EmbedProvider should be of the form "EmbedProvider ..." (with optional leading article) (golint)
- Line 207: warning: comment on exported type EmbedAuthor should be of the form "EmbedAuthor ..." (with optional leading article) (golint)
- Line 215: warning: comment on exported type EmbedFooter should be of the form "EmbedFooter ..." (with optional leading article) (golint)
- Line 222: warning: comment on exported type EmbedField should be of the form "EmbedField ..." (with optional leading article) (golint)
-
-
vvgo/pkg/api/server.go
- Line 11: warning: exported type Server should have comment or be unexported (golint)
- Line 15: warning: exported function NewServer should have comment or be unexported (golint)
-
-
vvgo/pkg/api/version.go
- Line 8: warning: exported function Version should have comment or be unexported (golint)
-
-
vvgo/pkg/log/log.go
- Line 9: warning: exported function Logger should have comment or be unexported (golint)
- Line 22: warning: exported function StdLogger should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/credits.go
- Line 10: warning: exported type Credit should have comment or be unexported (golint)
- Line 19: warning: exported type Credits should have comment or be unexported (golint)
- Line 21: warning: exported function ListCredits should have comment or be unexported (golint)
- Line 45: warning: exported method Credits.Sort should have comment or be unexported (golint)
- Line 47: warning: exported method Credits.ForProject should have comment or be unexported (golint)
- Line 57: warning: exported method Credits.WebsitePasta should have comment or be unexported (golint)
- Line 66: warning: exported method Credits.VideoPasta should have comment or be unexported (golint)
- Line 84: warning: exported method Credits.YoutubePasta should have comment or be unexported (golint)
-
-
vvgo/pkg/discord/client.go
- Line 20: warning: exported var ErrNon200Response should have comment or be unexported (golint)
- Line 21: warning: exported var ErrInvalidOAuthCode should have comment or be unexported (golint)
- Line 23: warning: exported const ClientPublicKey should have comment or be unexported (golint)
- Line 24: warning: exported const ApplicationID should have comment or be unexported (golint)
- Line 25: warning: exported const OAuthClientID should have comment or be unexported (golint)
- Line 26: warning: exported const VVGOGuildID should have comment or be unexported (golint)
- Line 27: warning: exported const VVGOVerifiedMemberRoleID should have comment or be unexported (golint)
- Line 28: warning: exported const VVGOProductionTeamRoleID should have comment or be unexported (golint)
- Line 29: warning: exported const VVGOExecutiveDirectorRoleID should have comment or be unexported (golint)
- Line 36: warning: exported function NewClient should have comment or be unexported (golint)
- Line 68: warning: exported method Client.LoginURL should have comment or be unexported (golint)
- Line 78: warning: comment on exported method Client.QueryOAuth should be of the form "QueryOAuth ..." (golint)
- Line 116: warning: comment on exported method Client.QueryIdentity should be of the form "QueryIdentity ..." (golint)
- Line 143: warning: comment on exported method Client.QueryGuildMember should be of the form "QueryGuildMember ..." (golint)
- Line 161: warning: exported method Client.GetApplicationCommands should have comment or be unexported (golint)
- Line 172: warning: exported method Client.CreateApplicationCommand should have comment or be unexported (golint)
- Line 192: warning: exported method Client.DeleteApplicationCommand should have comment or be unexported (golint)
- Line 202: warning: exported method Client.CreateMessage should have comment or be unexported (golint)
- Line 220: warning: exported method Client.EditMessage should have comment or be unexported (golint)
-
-
vvgo/pkg/api/download.go
- Line 10: warning: exported const ProtectedLinkExpiry should have comment or be unexported (golint)
- Line 12: warning: exported type DownloadConfig should have comment or be unexported (golint)
- Line 16: warning: exported var DownloadHandler should have comment or be unexported (golint)
-
-
vvgo/pkg/api/projects.go
- Line 9: warning: exported function ProjectsApi should have comment or be unexported (golint)
- Line 31: warning: exported function ProjectsView should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/leaders.go
- Line 7: warning: exported type Leaders should have comment or be unexported (golint)
- Line 9: warning: exported type Leader should have comment or be unexported (golint)
- Line 18: warning: exported function ListLeaders should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/sheets.go
- Line 16: warning: exported const CacheTTL should have comment or be unexported (golint)
- Line 20: warning: exported function WebsiteDataSpreadsheetID should have comment or be unexported (golint)
- Line 26: warning: exported function ReadSheet should have comment or be unexported (golint)
- Line 75: warning: exported function WriteValuesToRedis should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/submissions.go
- Line 9: warning: exported type Submission should have comment or be unexported (golint)
- Line 15: warning: exported type Submissions should have comment or be unexported (golint)
- Line 17: warning: exported function ListSubmissions should have comment or be unexported (golint)
- Line 38: warning: exported method Submissions.ToCredits should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/skyward_sword_intent.go
- Line 8: warning: exported const SkywardSwordIntentID should have comment or be unexported (golint)
- Line 10: warning: exported type SkywardSwordIntent should have comment or be unexported (golint)
- Line 17: warning: exported type SkywardSwordIntents should have comment or be unexported (golint)
- Line 19: warning: exported function ListSkywardSwordIntents should have comment or be unexported (golint)
- Line 43: warning: exported method SkywardSwordIntents.Sort should have comment or be unexported (golint)
-
-
vvgo/pkg/api/roles.go
- Line 8: warning: exported function RolesApi should have comment or be unexported (golint)
-
-
vvgo/pkg/redis/redis.go
- Line 10: warning: exported type Client should have comment or be unexported (golint)
- Line 15: warning: exported type Config should have comment or be unexported (golint)
- Line 23: warning: exported function Initialize should have comment or be unexported (golint)
- Line 27: warning: exported function InitializeFromEnv should have comment or be unexported (golint)
- Line 33: warning: exported function Do should have comment or be unexported (golint)
- Line 37: warning: exported function NewClient should have comment or be unexported (golint)
- Line 56: warning: exported function NewClientMust should have comment or be unexported (golint)
- Line 64: warning: exported type Action should have comment or be unexported (golint)
- Line 70: warning: exported function Cmd should have comment or be unexported (golint)
- Line 78: warning: exported method Client.Do should have comment or be unexported (golint)
-
-
vvgo/pkg/parse_config/config.go
- Line 1: warning: don't use an underscore in package name (golint)
- Line 12: warning: exported var Namespace should have comment or be unexported (golint)
- Line 16: warning: exported function UseTestNamespace should have comment or be unexported (golint)
- Line 20: warning: exported function WriteRedisHashValue should have comment or be unexported (golint)
- Line 24: warning: exported function WriteToRedisHash should have comment or be unexported (golint)
- Line 67: warning: exported function ReadRedisHashValue should have comment or be unexported (golint)
- Line 71: warning: exported function ReadFromRedisHash should have comment or be unexported (golint)
- Line 98: warning: exported function SetDefaults should have comment or be unexported (golint)
-
-
vvgo/pkg/sheets/query.go
- Line 5: warning: exported type Query should have comment or be unexported (golint)
- Line 7: warning: exported function NewQuery should have comment or be unexported (golint)
- Line 9: warning: exported method Query.WithField should have comment or be unexported (golint)
- Line 10: warning: exported method Query.WithFieldsUnsafe should have comment or be unexported (golint)
- Line 41: warning: exported method Query.MatchStruct should have comment or be unexported (golint)
-
-
vvgo/pkg/api/parts.go
- Line 9: warning: exported var PartsView should have comment or be unexported (golint)
- Line 11: warning: exported function PartsApi should have comment or be unexported (golint)
-
-
vvgo/pkg/error_wrappers/error_wrappers.go
- Line 1: warning: don't use an underscore in package name (golint)
- Line 8: warning: exported function NewRequestFailed should have comment or be unexported (golint)
- Line 12: warning: exported function HTTPDoFailed should have comment or be unexported (golint)
- Line 16: warning: exported function Non200StatusCode should have comment or be unexported (golint)
-
-
vvgo/pkg/login/sessions.go
- Line 19: warning: exported var ErrSessionNotFound should have comment or be unexported (golint)
- Line 29: warning: exported type Config should have comment or be unexported (golint)
- Line 53: warning: exported method Store.Config should have comment or be unexported (golint)
- Line 64: warning: exported method Store.DeleteSessionFromRequest should have comment or be unexported (golint)
- Line 89: warning: exported function NewCookieValue should have comment or be unexported (golint)
-
-
vvgo/pkg/api/index.go
- Line 8: warning: exported var IndexView should have comment or be unexported (golint)
- Line 9: warning: exported var ContactUs should have comment or be unexported (golint)
- Line 11: warning: exported var AboutView should have comment or be unexported (golint)
- Line 23: warning: exported function ServeTemplate should have comment or be unexported (golint)
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!
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!