Preparing report...

Report for github.com/ccbrown/api-fu

A+    Excellent!    Found 54 issues across 111 files

Tweet

gofmt98%

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


go_vet100%

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

No problems detected. Good job!


gocyclo80%

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.

    • api-fu/graphqlws/connection.go
    • Line 235: warning: cyclomatic complexity 20 of function (*Connection).writeLoop() is high (> 15) (gocyclo)
    • Line 144: warning: cyclomatic complexity 16 of function (*Connection).handleMessage() is high (> 15) (gocyclo)
    • api-fu/pagination.go
    • Line 150: warning: cyclomatic complexity 23 of function Connection() is high (> 15) (gocyclo)
    • Line 424: warning: cyclomatic complexity 18 of function TimeBasedConnection() is high (> 15) (gocyclo)
    • api-fu/graphql/validator/validate_fields.go
    • Line 11: warning: cyclomatic complexity 26 of function validateFields() is high (> 15) (gocyclo)
    • Line 210: warning: cyclomatic complexity 25 of function validateSameResponseShape() is high (> 15) (gocyclo)
    • Line 159: warning: cyclomatic complexity 25 of function valuesAreIdentical() is high (> 15) (gocyclo)
    • Line 106: warning: cyclomatic complexity 19 of function validateFieldsInSetCanMerge() is high (> 15) (gocyclo)
    • api-fu/graphql/executor/executor.go
    • Line 342: warning: cyclomatic complexity 21 of function (*executor).completeValue() is high (> 15) (gocyclo)
    • Line 439: warning: cyclomatic complexity 21 of function (*executor).collectFieldsImpl() is high (> 15) (gocyclo)

golint65%

Golint is a linter for Go source code.

    • api-fu/graphql/scanner/scanner.go
    • Line 10: warning: exported type Error should have comment or be unexported (golint)
    • Line 20: warning: exported type Scanner should have comment or be unexported (golint)
    • Line 38: warning: exported type Mode should have comment or be unexported (golint)
    • Line 41: warning: exported const ScanIgnored should have comment (or a comment on this block) or be unexported (golint)
    • Line 44: warning: exported function New should have comment or be unexported (golint)
    • Line 55: warning: exported method Scanner.Errors should have comment or be unexported (golint)
    • Line 121: warning: exported method Scanner.Scan should have comment or be unexported (golint)
    • Line 207: warning: exported method Scanner.Token should have comment or be unexported (golint)
    • Line 211: warning: exported method Scanner.Position should have comment or be unexported (golint)
    • Line 215: warning: exported method Scanner.Literal should have comment or be unexported (golint)
    • Line 219: warning: exported method Scanner.StringValue should have comment or be unexported (golint)
    • Line 222: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • api-fu/examples/chat/app/user.go
    • Line 8: warning: exported method Session.CreateUser should have comment or be unexported (golint)
    • Line 26: warning: exported method Session.GetUsersByIds should have comment or be unexported (golint)
    • api-fu/graphql/schema/introspection/schema_data.go
    • Line 9: warning: exported type SchemaData should have comment or be unexported (golint)
    • Line 17: warning: comment on exported method SchemaData.GetSchemaDefinition should be of the form "GetSchemaDefinition ..." (golint)
    • Line 97: 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 124: 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 149: 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 161: 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 171: 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 179: warning: exported type DirectiveData should have comment or be unexported (golint)
    • Line 222: 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 229: warning: exported type TypeData should have comment or be unexported (golint)
    • Line 267: warning: exported type FieldData should have comment or be unexported (golint)
    • Line 290: 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 297: warning: exported type InputValueData should have comment or be unexported (golint)
    • Line 314: warning: exported type EnumValueData should have comment or be unexported (golint)
    • api-fu/graphql/schema/enum_type.go
    • Line 9: warning: exported type EnumType should have comment or be unexported (golint)
    • Line 16: warning: exported type EnumValueDefinition should have comment or be unexported (golint)
    • Line 27: warning: exported method EnumType.IsInputType should have comment or be unexported (golint)
    • Line 31: warning: exported method EnumType.IsOutputType should have comment or be unexported (golint)
    • Line 35: warning: exported method EnumType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 39: warning: exported method EnumType.IsSameType should have comment or be unexported (golint)
    • Line 43: warning: exported method EnumType.TypeName should have comment or be unexported (golint)
    • Line 50: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 60: warning: exported method EnumType.CoerceVariableValue should have comment or be unexported (golint)
    • Line 69: warning: exported method EnumType.CoerceLiteral should have comment or be unexported (golint)
    • Line 78: warning: exported method EnumType.CoerceResult should have comment or be unexported (golint)
    • Line 87: warning: exported function IsEnumType should have comment or be unexported (golint)
    • api-fu/graphql/schema/union_type.go
    • Line 5: warning: exported type UnionType should have comment or be unexported (golint)
    • Line 16: warning: exported method UnionType.IsInputType should have comment or be unexported (golint)
    • Line 20: warning: exported method UnionType.IsOutputType should have comment or be unexported (golint)
    • Line 24: warning: exported method UnionType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 28: warning: exported method UnionType.IsSameType should have comment or be unexported (golint)
    • Line 32: warning: exported method UnionType.TypeName should have comment or be unexported (golint)
    • api-fu/graphql/schema/scalar_type.go
    • Line 9: warning: exported type ScalarType should have comment or be unexported (golint)
    • Line 29: warning: exported method ScalarType.IsInputType should have comment or be unexported (golint)
    • Line 33: warning: exported method ScalarType.IsOutputType should have comment or be unexported (golint)
    • Line 37: warning: exported method ScalarType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 41: warning: exported method ScalarType.IsSameType should have comment or be unexported (golint)
    • Line 45: warning: exported method ScalarType.TypeName should have comment or be unexported (golint)
    • Line 49: warning: exported method ScalarType.CoerceVariableValue should have comment or be unexported (golint)
    • Line 56: warning: exported method ScalarType.CoerceResult should have comment or be unexported (golint)
    • Line 63: warning: exported function IsScalarType should have comment or be unexported (golint)
    • api-fu/examples/chat/store/user.go
    • Line 9: warning: exported var ErrUserHandleExists should have comment or be unexported (golint)
    • Line 11: warning: comment on exported method Store.AddUser should be of the form "AddUser ..." (golint)
    • Line 29: warning: exported method Store.GetUsersByIds should have comment or be unexported (golint)
    • Line 34: warning: exported method Store.GetUserByHandle should have comment or be unexported (golint)
    • api-fu/examples/chat/app/message.go
    • Line 9: warning: exported method Session.CreateMessage should have comment or be unexported (golint)
    • Line 33: warning: exported method Session.GetMessagesByIds should have comment or be unexported (golint)
    • api-fu/cmd/gql-client-gen/main.go
    • Line 317: warning: exported function Generate should have comment or be unexported (golint)
    • Line 356: warning: exported function LoadSchema should have comment or be unexported (golint)
    • Line 380: warning: exported function Run should have comment or be unexported (golint)
    • api-fu/graphql/schema/object_type.go
    • Line 8: warning: exported type ObjectType should have comment or be unexported (golint)
    • Line 25: warning: exported method ObjectType.IsInputType should have comment or be unexported (golint)
    • Line 29: warning: exported method ObjectType.IsOutputType should have comment or be unexported (golint)
    • Line 33: warning: exported method ObjectType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 52: warning: exported method ObjectType.IsSameType should have comment or be unexported (golint)
    • Line 56: warning: exported method ObjectType.TypeName should have comment or be unexported (golint)
    • Line 107: warning: exported function IsObjectType should have comment or be unexported (golint)
    • api-fu/graphql/ast/ast.go
    • Line 5: warning: exported type Node should have comment or be unexported (golint)
    • Line 9: warning: exported type Document should have comment or be unexported (golint)
    • Line 13: warning: exported method Document.Position should have comment or be unexported (golint)
    • Line 15: warning: comment on exported type Definition should be of the form "Definition ..." (with optional leading article) (golint)
    • Line 20: warning: exported type OperationDefinition should have comment or be unexported (golint)
    • Line 28: warning: exported method OperationDefinition.Position should have comment or be unexported (golint)
    • Line 35: warning: exported type OperationType should have comment or be unexported (golint)
    • Line 40: warning: exported method OperationType.Position should have comment or be unexported (golint)
    • Line 42: warning: exported type FragmentDefinition should have comment or be unexported (golint)
    • Line 50: warning: exported method FragmentDefinition.Position should have comment or be unexported (golint)
    • Line 52: warning: exported type VariableDefinition should have comment or be unexported (golint)
    • Line 58: warning: exported method VariableDefinition.Position should have comment or be unexported (golint)
    • Line 60: warning: comment on exported type Type should be of the form "Type ..." (with optional leading article) (golint)
    • Line 65: warning: exported type ListType should have comment or be unexported (golint)
    • Line 71: warning: exported method ListType.Position should have comment or be unexported (golint)
    • Line 73: warning: exported type NonNullType should have comment or be unexported (golint)
    • Line 77: warning: exported method NonNullType.Position should have comment or be unexported (golint)
    • Line 79: warning: exported type Directive should have comment or be unexported (golint)
    • Line 85: warning: exported method Directive.Position should have comment or be unexported (golint)
    • Line 87: warning: exported type SelectionSet should have comment or be unexported (golint)
    • Line 93: warning: exported method SelectionSet.Position should have comment or be unexported (golint)
    • Line 95: warning: comment on exported type Selection should be of the form "Selection ..." (with optional leading article) (golint)
    • Line 101: warning: exported type Field should have comment or be unexported (golint)
    • Line 109: warning: exported method Field.Position should have comment or be unexported (golint)
    • Line 116: warning: exported method Field.SelectionDirectives should have comment or be unexported (golint)
    • Line 116: warning: receiver name s should be consistent with previous receiver name n for Field (golint)
    • Line 118: warning: exported type FragmentSpread should have comment or be unexported (golint)
    • Line 124: warning: exported method FragmentSpread.Position should have comment or be unexported (golint)
    • Line 125: warning: exported method FragmentSpread.SelectionDirectives should have comment or be unexported (golint)
    • Line 125: warning: receiver name s should be consistent with previous receiver name n for FragmentSpread (golint)
    • Line 127: warning: exported type InlineFragment should have comment or be unexported (golint)
    • Line 134: warning: exported method InlineFragment.Position should have comment or be unexported (golint)
    • Line 135: warning: exported method InlineFragment.SelectionDirectives should have comment or be unexported (golint)
    • Line 135: warning: receiver name s should be consistent with previous receiver name n for InlineFragment (golint)
    • Line 137: warning: exported type Argument should have comment or be unexported (golint)
    • Line 142: warning: exported method Argument.Position should have comment or be unexported (golint)
    • Line 144: warning: exported type Name should have comment or be unexported (golint)
    • Line 149: warning: exported method Name.Position should have comment or be unexported (golint)
    • Line 151: warning: exported type NamedType should have comment or be unexported (golint)
    • Line 155: warning: exported method NamedType.Position should have comment or be unexported (golint)
    • Line 157: warning: comment on exported type Value should be of the form "Value ..." (with optional leading article) (golint)
    • Line 164: warning: exported type Variable should have comment or be unexported (golint)
    • Line 169: warning: exported method Variable.IsValue should have comment or be unexported (golint)
    • Line 170: warning: exported method Variable.Position should have comment or be unexported (golint)
    • Line 172: warning: exported type BooleanValue should have comment or be unexported (golint)
    • Line 177: warning: exported method BooleanValue.IsValue should have comment or be unexported (golint)
    • Line 178: warning: exported method BooleanValue.Position should have comment or be unexported (golint)
    • Line 180: warning: exported type FloatValue should have comment or be unexported (golint)
    • Line 185: warning: exported method FloatValue.IsValue should have comment or be unexported (golint)
    • Line 186: warning: exported method FloatValue.Position should have comment or be unexported (golint)
    • Line 188: warning: exported type IntValue should have comment or be unexported (golint)
    • Line 193: warning: exported method IntValue.IsValue should have comment or be unexported (golint)
    • Line 194: warning: exported method IntValue.Position should have comment or be unexported (golint)
    • Line 196: warning: exported type StringValue should have comment or be unexported (golint)
    • Line 203: warning: exported method StringValue.IsValue should have comment or be unexported (golint)
    • Line 204: warning: exported method StringValue.Position should have comment or be unexported (golint)
    • Line 206: warning: exported type EnumValue should have comment or be unexported (golint)
    • Line 211: warning: exported method EnumValue.IsValue should have comment or be unexported (golint)
    • Line 212: warning: exported method EnumValue.Position should have comment or be unexported (golint)
    • Line 214: warning: exported type NullValue should have comment or be unexported (golint)
    • Line 218: warning: exported method NullValue.IsValue should have comment or be unexported (golint)
    • Line 219: warning: exported method NullValue.Position should have comment or be unexported (golint)
    • Line 221: warning: exported function IsNullValue should have comment or be unexported (golint)
    • Line 226: warning: exported type ListValue should have comment or be unexported (golint)
    • Line 232: warning: exported method ListValue.IsValue should have comment or be unexported (golint)
    • Line 233: warning: exported method ListValue.Position should have comment or be unexported (golint)
    • Line 235: warning: exported type ObjectValue should have comment or be unexported (golint)
    • Line 241: warning: exported method ObjectValue.IsValue should have comment or be unexported (golint)
    • Line 242: warning: exported method ObjectValue.Position should have comment or be unexported (golint)
    • Line 244: warning: exported type ObjectField should have comment or be unexported (golint)
    • Line 249: warning: exported method ObjectField.Position should have comment or be unexported (golint)
    • api-fu/examples/chat/model/user.go
    • Line 10: warning: exported type User should have comment or be unexported (golint)
    • Line 18: warning: exported function PasswordHash should have comment or be unexported (golint)
    • Line 28: warning: exported function VerifyPasswordHash should have comment or be unexported (golint)
    • api-fu/examples/chat/app/error.go
    • Line 3: warning: exported type SanitizedError should have comment or be unexported (golint)
    • Line 8: warning: exported type InternalError should have comment or be unexported (golint)
    • Line 16: warning: exported method InternalError.SanitizedError should have comment or be unexported (golint)
    • Line 24: warning: exported method Session.InternalError should have comment or be unexported (golint)
    • Line 34: warning: exported type UserError should have comment or be unexported (golint)
    • Line 42: warning: exported method UserError.SanitizedError should have comment or be unexported (golint)
    • Line 46: warning: exported method Session.UserError should have comment or be unexported (golint)
    • Line 52: warning: exported method Session.AuthorizationError should have comment or be unexported (golint)
    • api-fu/examples/chat/app/session.go
    • Line 9: warning: exported type Session should have comment or be unexported (golint)
    • Line 15: warning: exported method App.NewSession should have comment or be unexported (golint)
    • Line 22: warning: exported method Session.WithHandleAndPassword should have comment or be unexported (golint)
    • api-fu/graphql/validator/validator.go
    • Line 10: warning: exported type Location should have comment or be unexported (golint)
    • Line 15: warning: exported type Error should have comment or be unexported (golint)
    • Line 65: warning: exported function ValidateDocument should have comment or be unexported (golint)
    • api-fu/graphql/schema/interface_type.go
    • Line 8: warning: exported type InterfaceType should have comment or be unexported (golint)
    • Line 19: warning: exported method InterfaceType.IsInputType should have comment or be unexported (golint)
    • Line 23: warning: exported method InterfaceType.IsOutputType should have comment or be unexported (golint)
    • Line 27: warning: exported method InterfaceType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 31: warning: exported method InterfaceType.IsSameType should have comment or be unexported (golint)
    • Line 35: warning: exported method InterfaceType.TypeName should have comment or be unexported (golint)
    • Line 42: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • api-fu/graphql/schema/schema.go
    • Line 11: warning: exported type Schema should have comment or be unexported (golint)
    • Line 21: warning: exported method Schema.QueryType should have comment or be unexported (golint)
    • Line 25: warning: exported method Schema.MutationType should have comment or be unexported (golint)
    • Line 29: warning: exported method Schema.SubscriptionType should have comment or be unexported (golint)
    • Line 33: warning: exported method Schema.Directives should have comment or be unexported (golint)
    • Line 37: warning: exported method Schema.NamedTypes should have comment or be unexported (golint)
    • Line 41: warning: exported method Schema.InterfaceImplementations should have comment or be unexported (golint)
    • Line 51: warning: exported function New should have comment or be unexported (golint)
    • Line 115: warning: exported type SchemaDefinition should have comment or be unexported (golint)
    • Line 128: warning: exported type Argument should have comment or be unexported (golint)
    • Line 133: warning: exported type Type should have comment or be unexported (golint)
    • Line 141: warning: exported type NamedType should have comment or be unexported (golint)
    • Line 146: warning: exported type WrappedType should have comment or be unexported (golint)
    • Line 151: warning: exported function UnwrappedType should have comment or be unexported (golint)
    • Line 165: warning: exported function CoerceVariableValue should have comment or be unexported (golint)
    • Line 193: warning: exported function CoerceLiteral should have comment or be unexported (golint)
    • api-fu/graphql/token/token.go
    • Line 3: warning: exported type Token should have comment or be unexported (golint)
    • Line 6: warning: exported const INVALID should have comment (or a comment on this block) or be unexported (golint)
    • Line 21: warning: comment on exported method Token.IsIgnored should be of the form "IsIgnored ..." (golint)
    • Line 31: warning: exported type Position should have comment or be unexported (golint)
    • api-fu/graphql/schema/directive.go
    • Line 8: warning: exported type DirectiveLocation should have comment or be unexported (golint)
    • Line 11: warning: exported const DirectiveLocationQuery should have comment (or a comment on this block) or be unexported (golint)
    • Line 32: warning: exported type DirectiveDefinition should have comment or be unexported (golint)
    • Line 74: warning: exported type Directive should have comment or be unexported (golint)
    • api-fu/graphqlws.go
    • Line 27: 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)
    • api-fu/graphql/schema/nonnull_type.go
    • Line 5: warning: exported type NonNullType should have comment or be unexported (golint)
    • Line 9: warning: exported function NewNonNullType should have comment or be unexported (golint)
    • Line 19: warning: exported method NonNullType.IsInputType should have comment or be unexported (golint)
    • Line 23: warning: exported method NonNullType.IsOutputType should have comment or be unexported (golint)
    • Line 27: warning: exported method NonNullType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 34: warning: exported method NonNullType.IsSameType should have comment or be unexported (golint)
    • Line 52: warning: exported function IsNonNullType should have comment or be unexported (golint)
    • Line 57: warning: exported function NullableType should have comment or be unexported (golint)
    • api-fu/examples/chat/api/api.go
    • Line 15: warning: exported function DeserializeId should have comment or be unexported (golint)
    • Line 40: warning: exported type API should have comment or be unexported (golint)
    • Line 57: warning: exported method API.ServeGraphQL should have comment or be unexported (golint)
    • Line 62: warning: exported method API.ServeGraphQLWS should have comment or be unexported (golint)
    • api-fu/graphql/schema/list_type.go
    • Line 9: warning: exported type ListType should have comment or be unexported (golint)
    • Line 13: warning: exported function NewListType should have comment or be unexported (golint)
    • Line 23: warning: exported method ListType.IsInputType should have comment or be unexported (golint)
    • Line 27: warning: exported method ListType.IsOutputType should have comment or be unexported (golint)
    • Line 31: warning: exported method ListType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 35: warning: exported method ListType.IsSameType should have comment or be unexported (golint)
    • Line 46: warning: exported method ListType.CoerceVariableValue should have comment or be unexported (golint)
    • Line 56: 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 64: 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 71: warning: exported method ListType.CoerceLiteral should have comment or be unexported (golint)
    • Line 81: 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 89: 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 96: warning: exported function IsListType should have comment or be unexported (golint)
    • api-fu/graphql/parser/parser.go
    • Line 11: warning: exported type Location should have comment or be unexported (golint)
    • Line 16: warning: exported type Error should have comment or be unexported (golint)
    • Line 25: warning: exported function ParseDocument should have comment or be unexported (golint)
    • Line 39: warning: exported function ParseValue should have comment or be unexported (golint)
    • api-fu/examples/chat/app/channel.go
    • Line 9: warning: exported method Session.CreateChannel should have comment or be unexported (golint)
    • Line 29: warning: exported method Session.GetChannelById should have comment or be unexported (golint)
    • Line 37: warning: exported method Session.GetChannelsByIds should have comment or be unexported (golint)
    • Line 42: warning: exported method Session.GetChannels should have comment or be unexported (golint)
    • api-fu/graphql/schema/introspection/introspection.go
    • Line 9: warning: exported var NamedTypes should have comment or be unexported (golint)
    • Line 20: warning: exported var MetaFields should have comment or be unexported (golint)
    • Line 63: warning: exported var SchemaType should have comment or be unexported (golint)
    • Line 129: warning: exported var TypeKindType should have comment or be unexported (golint)
    • Line 159: warning: exported var TypeType should have comment or be unexported (golint)
    • Line 323: warning: exported var DirectiveLocationType should have comment or be unexported (golint)
    • Line 383: warning: exported var DirectiveType should have comment or be unexported (golint)
    • Line 418: warning: exported var FieldType should have comment or be unexported (golint)
    • Line 465: warning: exported var EnumValueType should have comment or be unexported (golint)
    • Line 500: warning: exported var InputValueType should have comment or be unexported (golint)
    • api-fu/graphql/schema/input_object_type.go
    • Line 10: warning: exported type InputObjectType should have comment or be unexported (golint)
    • Line 34: warning: exported method InputObjectType.IsInputType should have comment or be unexported (golint)
    • Line 38: warning: exported method InputObjectType.IsOutputType should have comment or be unexported (golint)
    • Line 42: warning: exported method InputObjectType.IsSubTypeOf should have comment or be unexported (golint)
    • Line 46: warning: exported method InputObjectType.IsSameType should have comment or be unexported (golint)
    • Line 50: warning: exported method InputObjectType.TypeName should have comment or be unexported (golint)
    • Line 54: warning: exported method InputObjectType.CoerceVariableValue 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 (move short variable declaration to its own line if necessary) (golint)
    • Line 91: warning: exported method InputObjectType.CoerceLiteral should have comment or be unexported (golint)
    • Line 98: 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 106: 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 132: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • api-fu/examples/chat/store/channel.go
    • Line 7: warning: exported method Store.AddChannel should have comment or be unexported (golint)
    • Line 20: warning: exported method Store.GetChannelsByIds should have comment or be unexported (golint)
    • Line 25: warning: exported method Store.GetChannels should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!