Preparing report...

Report for github.com/MessageDream/goby

A    Great!    Found 58 issues across 58 files

Tweet

gofmt94%

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!


gocyclo87%

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.

    • goby/module/infrastructure/tool.go
    • Line 240: warning: cyclomatic complexity 18 of function timeSince() is high (> 15) (gocyclo)
    • Line 162: warning: cyclomatic complexity 16 of function computeTimeDiff() is high (> 15) (gocyclo)
    • Line 342: warning: cyclomatic complexity 16 of function Subtract() is high (> 15) (gocyclo)

golint0%

Golint is a linter for Go source code.

    • goby/router/auth/auth.go
    • Line 12: warning: exported function SignInGet should have comment or be unexported (golint)
    • Line 16: warning: exported function SignUpGet should have comment or be unexported (golint)
    • Line 20: warning: exported function SignOutGet should have comment or be unexported (golint)
    • Line 32: warning: exported function LinkGet should have comment or be unexported (golint)
    • Line 36: warning: exported function Authenticated should have comment or be unexported (golint)
    • Line 59: warning: exported function SignOutPost should have comment or be unexported (golint)
    • goby/model/dto/dto_package.go
    • Line 3: warning: exported type BlobInfo should have comment or be unexported (golint)
    • Line 8: warning: exported type PackageInfo should have comment or be unexported (golint)
    • Line 18: warning: exported type Package should have comment or be unexported (golint)
    • Line 30: warning: exported type PackageMetrics should have comment or be unexported (golint)
    • Line 38: warning: exported type UpdatePackageInfo should have comment or be unexported (golint)
    • goby/module/template/template.go
    • Line 26: warning: exported function InitTemplate should have comment or be unexported (golint)
    • Line 33: warning: exported function Str2html should have comment or be unexported (golint)
    • Line 37: warning: exported function Range should have comment or be unexported (golint)
    • Line 41: warning: exported function List should have comment or be unexported (golint)
    • Line 54: warning: exported function ShortSha should have comment or be unexported (golint)
    • Line 65: warning: exported function NewFuncMap should have comment or be unexported (golint)
    • Line 137: warning: exported function Oauth2Icon should have comment or be unexported (golint)
    • Line 153: warning: exported function Oauth2Name should have comment or be unexported (golint)
    • goby/model/user.go
    • Line 23: warning: exported const USER_STATUS_UN_ACTIVE should have comment (or a comment on this block) or be unexported (golint)
    • Line 28: warning: exported type User should have comment or be unexported (golint)
    • Line 43: warning: exported method User.EncodePasswd should have comment or be unexported (golint)
    • Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 48: warning: exported method User.ValidatePassword should have comment or be unexported (golint)
    • Line 48: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 54: warning: exported method User.GenerateRands should have comment or be unexported (golint)
    • Line 54: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 58: warning: exported method User.GenerateSalt should have comment or be unexported (golint)
    • Line 58: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 62: warning: exported method User.Tokens should have comment or be unexported (golint)
    • Line 62: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 68: warning: exported method User.Collaborators should have comment or be unexported (golint)
    • Line 68: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 74: warning: exported method User.Apps should have comment or be unexported (golint)
    • Line 74: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 80: warning: exported method User.Update should have comment or be unexported (golint)
    • Line 80: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 84: warning: exported method User.Create should have comment or be unexported (golint)
    • Line 84: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 94: warning: exported method User.Get should have comment or be unexported (golint)
    • Line 94: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 99: warning: exported method User.Delete should have comment or be unexported (golint)
    • Line 99: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 107: warning: exported method User.Exist should have comment or be unexported (golint)
    • Line 107: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 112: warning: exported method User.EmailUsable should have comment or be unexported (golint)
    • Line 112: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 123: warning: exported method User.Convert should have comment or be unexported (golint)
    • Line 123: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 131: warning: exported function FindUserByIDs should have comment or be unexported (golint)
    • Line 136: warning: exported function QueryUsers should have comment or be unexported (golint)
    • Line 175: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 179: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 183: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 187: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 190: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 199: warning: exported function NewMailerUser should have comment or be unexported (golint)
    • goby/module/infrastructure/infrastructure.go
    • Line 8: warning: exported type TplName should have comment or be unexported (golint)
    • Line 10: warning: exported type ApiJsonError should have comment or be unexported (golint)
    • Line 15: warning: exported type IntentError should have comment or be unexported (golint)
    • Line 22: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 26: warning: exported function MakeIntentError should have comment or be unexported (golint)
    • Line 66: warning: exported function InitInfrastructure should have comment or be unexported (golint)
    • goby/model/dto/dto_client.go
    • Line 3: warning: exported type UpdateCheck should have comment or be unexported (golint)
    • Line 12: warning: exported type UpdateCheckCache should have comment or be unexported (golint)
    • goby/core/appService/app_service.go
    • Line 1: warning: don't use MixedCaps in package name; appService should be appservice (golint)
    • Line 10: warning: should not use dot imports (golint)
    • Line 14: warning: should not use dot imports (golint)
    • Line 17: warning: exported function List should have comment or be unexported (golint)
    • Line 95: warning: exported function Create should have comment or be unexported (golint)
    • Line 165: warning: exported function Delete should have comment or be unexported (golint)
    • Line 189: warning: exported function ChangeName should have comment or be unexported (golint)
    • Line 216: warning: exported function Transfer should have comment or be unexported (golint)
    • goby/router/web/access_key.go
    • Line 10: warning: exported const ACCESSKEYS should have comment (or a comment on this block) or be unexported (golint)
    • Line 13: warning: exported function AccessKeysGet should have comment or be unexported (golint)
    • goby/module/infrastructure/tool.go
    • Line 26: warning: exported function EncodeMd5 should have comment or be unexported (golint)
    • Line 32: warning: exported function EncodeSHA256 should have comment or be unexported (golint)
    • Line 38: warning: exported function FileMd5 should have comment or be unexported (golint)
    • Line 54: warning: exported function FileSHA256 should have comment or be unexported (golint)
    • Line 69: warning: exported function BasicAuthDecode should have comment or be unexported (golint)
    • Line 79: warning: exported function BasicAuthEncode should have comment or be unexported (golint)
    • Line 83: warning: exported function GetRandomString should have comment or be unexported (golint)
    • Line 97: warning: exported function VerifyEmail should have comment or be unexported (golint)
    • Line 102: warning: exported function VerifyTimeLimitCode should have comment or be unexported (golint)
    • Line 125: warning: exported const TimeLimitCodeLength should have comment or be unexported (golint)
    • Line 127: warning: exported function CreateTimeLimitCode should have comment or be unexported (golint)
    • Line 154: warning: exported const Minute should have comment (or a comment on this block) or be unexported (golint)
    • Line 220: warning: exported function TimeSincePro should have comment or be unexported (golint)
    • Line 294: warning: exported function TimeSince should have comment or be unexported (golint)
    • Line 299: warning: exported const Byte should have comment (or a comment on this block) or be unexported (golint)
    • Line 337: warning: exported function FileSize should have comment or be unexported (golint)
    • Line 342: warning: exported function Subtract should have comment or be unexported (golint)
    • Line 386: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 431: warning: comment on exported function DateParse should be of the form "DateParse ..." (golint)
    • Line 438: warning: comment on exported function DateFormat should be of the form "DateFormat ..." (golint)
    • Line 445: warning: exported function FileExist should have comment or be unexported (golint)
    • Line 450: warning: exported function SaveFileToTemp should have comment or be unexported (golint)
    • Line 470: warning: exported function CopyDirFiles should have comment or be unexported (golint)
    • Line 511: warning: exported function Compress should have comment or be unexported (golint)
    • Line 583: warning: exported function DeCompress should have comment or be unexported (golint)
    • goby/core/collaboratorService/collaborator_service.go
    • Line 1: warning: don't use MixedCaps in package name; collaboratorService should be collaboratorservice (golint)
    • Line 6: warning: should not use dot imports (golint)
    • Line 11: warning: exported function List should have comment or be unexported (golint)
    • Line 21: warning: exported function Add should have comment or be unexported (golint)
    • Line 61: warning: exported function Delete should have comment or be unexported (golint)
    • goby/router/collaborator/collaborator.go
    • Line 8: warning: should not use dot imports (golint)
    • Line 11: warning: exported function Get should have comment or be unexported (golint)
    • Line 47: warning: exported function Post should have comment or be unexported (golint)
    • Line 58: warning: exported function Delete should have comment or be unexported (golint)
    • goby/core/deploymentService/deployment_service.go
    • Line 1: warning: don't use MixedCaps in package name; deploymentService should be deploymentservice (golint)
    • Line 12: warning: should not use dot imports (golint)
    • Line 15: warning: should not use dot imports (golint)
    • Line 23: warning: exported function GetDeployments should have comment or be unexported (golint)
    • Line 42: warning: exported function GetDeploymentMetrics should have comment or be unexported (golint)
    • Line 70: warning: exported function GetDeploymentHistories should have comment or be unexported (golint)
    • Line 103: warning: exported function DeleteDeploymentHistory should have comment or be unexported (golint)
    • Line 147: warning: exported function ReleaseDeployment should have comment or be unexported (golint)
    • Line 299: warning: exported function UpdateDeploymentPackage should have comment or be unexported (golint)
    • Line 372: warning: exported function AddDeployment should have comment or be unexported (golint)
    • Line 403: warning: exported function RenameDeployment should have comment or be unexported (golint)
    • Line 438: warning: exported function DeleteDeployment should have comment or be unexported (golint)
    • Line 463: warning: exported function RollbackDeployment should have comment or be unexported (golint)
    • Line 550: warning: exported function PromoteDeployment should have comment or be unexported (golint)
    • goby/module/form/form.go
    • Line 12: warning: exported type Form should have comment or be unexported (golint)
    • Line 20: warning: exported function AssignForm should have comment or be unexported (golint)
    • Line 50: warning: exported function GetSize should have comment or be unexported (golint)
    • Line 54: warning: exported function GetMinSize should have comment or be unexported (golint)
    • Line 58: warning: exported function GetMaxSize should have comment or be unexported (golint)
    • Line 62: warning: exported function GetInclude should have comment or be unexported (golint)
    • goby/core/deploymentService/package_service.go
    • Line 1: warning: don't use MixedCaps in package name; deploymentService should be deploymentservice (golint)
    • Line 21: warning: should not use dot imports (golint)
    • Line 23: warning: should not use dot imports (golint)
    • Line 31: warning: error var packageTypeIOS should have name of the form errFoo (golint)
    • Line 32: warning: error var packageTypeAndroid should have name of the form errFoo (golint)
    • Line 36: warning: exported const MANIFEST_FILE_NAME should have comment (or a comment on this block) or be unexported (golint)
    • goby/model/dto/dto_user.go
    • Line 5: warning: exported type UserToken should have comment or be unexported (golint)
    • Line 16: warning: exported type User should have comment or be unexported (golint)
    • Line 23: warning: exported type Collaborator should have comment or be unexported (golint)
    • Line 28: warning: exported type Pager should have comment or be unexported (golint)
    • Line 36: warning: exported type UserDetail should have comment or be unexported (golint)
    • goby/router/web/auth.go
    • Line 12: warning: should not use dot imports (golint)
    • Line 24: warning: exported const SIGNUP should have comment (or a comment on this block) or be unexported (golint)
    • Line 31: warning: exported function SignUpGet should have comment or be unexported (golint)
    • Line 45: warning: exported function SignUpPost should have comment or be unexported (golint)
    • Line 124: warning: exported function SignInGet should have comment or be unexported (golint)
    • Line 156: warning: exported function SignInPost should have comment or be unexported (golint)
    • Line 200: warning: exported function SignOutGet should have comment or be unexported (golint)
    • Line 212: warning: exported function ActivateGet should have comment or be unexported (golint)
    • Line 266: warning: exported function AutoSignIn should have comment or be unexported (golint)
    • Line 316: warning: exported function FindPwdGet should have comment or be unexported (golint)
    • Line 325: warning: exported function FindPwdPost should have comment or be unexported (golint)
    • Line 358: warning: exported function ResetPwdGet should have comment or be unexported (golint)
    • Line 377: warning: exported function ResetPwdPost should have comment or be unexported (golint)
    • goby/module/context/auth.go
    • Line 14: warning: exported type ToggleOptions should have comment or be unexported (golint)
    • Line 22: warning: exported const TOKEN_TYPE_AUTH should have comment (or a comment on this block) or be unexported (golint)
    • Line 26: warning: exported function IsAPIPath should have comment or be unexported (golint)
    • Line 44: warning: exported function APIToggle should have comment or be unexported (golint)
    • Line 69: warning: exported function Toggle should have comment or be unexported (golint)
    • goby/core/tokenService/token_service.go
    • Line 1: warning: don't use MixedCaps in package name; tokenService should be tokenservice (golint)
    • Line 6: warning: should not use dot imports (golint)
    • Line 11: warning: exported function List should have comment or be unexported (golint)
    • Line 19: warning: exported function Create should have comment or be unexported (golint)
    • Line 44: warning: exported function Update should have comment or be unexported (golint)
    • Line 72: warning: exported function DeleteByUIDAndName should have comment or be unexported (golint)
    • Line 85: warning: exported function Delete should have comment or be unexported (golint)
    • Line 98: warning: exported function AuthWithToken should have comment or be unexported (golint)
    • Line 110: warning: exported function CheckTokenSession should have comment or be unexported (golint)
    • goby/module/mailer/mail.go
    • Line 14: warning: exported const MAIL_AUTH_ACTIVATE should have comment (or a comment on this block) or be unexported (golint)
    • Line 31: warning: exported function InitMail should have comment or be unexported (golint)
    • Line 46: warning: exported type MailRender should have comment or be unexported (golint)
    • Line 50: warning: exported function SendTestMail should have comment or be unexported (golint)
    • Line 54: warning: exported type User should have comment or be unexported (golint)
    • Line 62: warning: exported function SendUserMail should have comment or be unexported (golint)
    • Line 81: warning: exported function SendActivateAccountMail should have comment or be unexported (golint)
    • Line 85: warning: exported function SendResetPasswordMail should have comment or be unexported (golint)
    • Line 89: warning: exported function SendActivateEmailMail should have comment or be unexported (golint)
    • Line 108: warning: exported function SendRegisterNotifyMail should have comment or be unexported (golint)
    • goby/router/install.go
    • Line 31: warning: exported const INSTALL should have comment (or a comment on this block) or be unexported (golint)
    • Line 94: warning: exported function InstallInit should have comment or be unexported (golint)
    • Line 115: warning: comment on exported function Install should be of the form "Install ..." (golint)
    • Line 155: warning: exported function InstallPost should have comment or be unexported (golint)
    • goby/router/web/admin.go
    • Line 13: warning: exported const ADMIN_USERS should have comment (or a comment on this block) or be unexported (golint)
    • Line 16: warning: exported function UsersGet should have comment or be unexported (golint)
    • Line 21: warning: exported function UsersQuery should have comment or be unexported (golint)
    • Line 37: warning: exported function UserAddPost should have comment or be unexported (golint)
    • Line 52: warning: exported function UserPatch should have comment or be unexported (golint)
    • goby/model/model.go
    • Line 13: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 16: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 20: warning: exported var ErrDataNotExist should have comment or be unexported (golint)
    • Line 40: warning: exported type JsonObjectConverter should have comment or be unexported (golint)
    • Line 44: warning: exported type ModelProtocol should have comment or be unexported (golint)
    • Line 56: warning: exported type DbCfg should have comment or be unexported (golint)
    • Line 63: warning: exported var HasEngine should have comment or be unexported (golint)
    • Line 94: warning: exported function InitDBConfig should have comment or be unexported (golint)
    • Line 159: warning: exported function NewTestEngine should have comment or be unexported (golint)
    • Line 169: warning: exported function SetEngine should have comment or be unexported (golint)
    • Line 191: warning: exported function NewEngine should have comment or be unexported (golint)
    • Line 207: warning: exported function Transaction should have comment or be unexported (golint)
    • Line 231: warning: exported function EngineGenerate should have comment or be unexported (golint)
    • Line 238: warning: exported function ModelCreate should have comment or be unexported (golint)
    • Line 247: warning: exported function ModelGet should have comment or be unexported (golint)
    • Line 254: warning: exported function ModelUpdate should have comment or be unexported (golint)
    • Line 268: warning: exported function Ping should have comment or be unexported (golint)
    • goby/model/package.go
    • Line 7: warning: exported type PackageDetail should have comment or be unexported (golint)
    • Line 12: warning: exported type Package should have comment or be unexported (golint)
    • Line 34: warning: exported method Package.Update should have comment or be unexported (golint)
    • Line 34: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 38: warning: exported method Package.Exist should have comment or be unexported (golint)
    • Line 38: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 43: warning: exported method Package.Create should have comment or be unexported (golint)
    • Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 47: warning: exported method Package.Get should have comment or be unexported (golint)
    • Line 47: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 52: warning: exported method Package.Delete should have comment or be unexported (golint)
    • Line 52: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 61: warning: exported method Package.FindPrePackages should have comment or be unexported (golint)
    • Line 61: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 77: warning: exported method Package.FindPackagesByVersionIDAndReleaseMethods should have comment or be unexported (golint)
    • Line 77: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 93: warning: exported type PackageDiff should have comment or be unexported (golint)
    • Line 104: warning: exported method PackageDiff.Update should have comment or be unexported (golint)
    • Line 104: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 108: warning: exported method PackageDiff.Exist should have comment or be unexported (golint)
    • Line 108: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 113: warning: exported method PackageDiff.Create should have comment or be unexported (golint)
    • Line 113: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 117: warning: exported method PackageDiff.Get should have comment or be unexported (golint)
    • Line 117: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 122: warning: exported method PackageDiff.Delete should have comment or be unexported (golint)
    • Line 122: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 131: warning: exported type PackageMetrics should have comment or be unexported (golint)
    • Line 143: warning: exported method PackageMetrics.Update should have comment or be unexported (golint)
    • Line 143: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 147: warning: exported method PackageMetrics.Exist should have comment or be unexported (golint)
    • Line 147: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 152: warning: exported method PackageMetrics.Create should have comment or be unexported (golint)
    • Line 152: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 156: warning: exported method PackageMetrics.Get should have comment or be unexported (golint)
    • Line 156: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 161: warning: exported method PackageMetrics.Delete should have comment or be unexported (golint)
    • Line 161: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 170: warning: exported type PackageMetricsDetail should have comment or be unexported (golint)
    • goby/router/home.go
    • Line 13: warning: exported const HOME should have comment (or a comment on this block) or be unexported (golint)
    • Line 17: warning: exported function Home should have comment or be unexported (golint)
    • Line 41: warning: exported function ReadMeGet should have comment or be unexported (golint)
    • Line 45: warning: exported function TokensGet should have comment or be unexported (golint)
    • Line 49: warning: exported function NotFound should have comment or be unexported (golint)
    • goby/model/deployment.go
    • Line 9: warning: exported type DeploymentDetail should have comment or be unexported (golint)
    • Line 16: warning: exported method DeploymentDetail.Convert should have comment or be unexported (golint)
    • Line 16: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 57: warning: exported type DeploymentHistoryDetail should have comment or be unexported (golint)
    • Line 63: warning: exported method DeploymentHistoryDetail.Convert should have comment or be unexported (golint)
    • Line 63: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 89: warning: exported type Deployment should have comment or be unexported (golint)
    • Line 102: warning: exported method Deployment.Update should have comment or be unexported (golint)
    • Line 102: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 106: warning: exported method Deployment.Exist should have comment or be unexported (golint)
    • Line 106: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 111: warning: exported method Deployment.Create should have comment or be unexported (golint)
    • Line 111: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 115: warning: exported method Deployment.Get should have comment or be unexported (golint)
    • Line 115: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 119: warning: exported method Deployment.Detail should have comment or be unexported (golint)
    • Line 119: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 138: warning: exported method Deployment.PackageOfVersion should have comment or be unexported (golint)
    • Line 138: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 159: warning: exported method Deployment.Histories should have comment or be unexported (golint)
    • Line 159: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 177: warning: exported method Deployment.PackageMetricsOfLabel should have comment or be unexported (golint)
    • Line 177: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 196: warning: exported method Deployment.GetPackageMetricsByAppIDAndName should have comment or be unexported (golint)
    • Line 196: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 215: warning: exported method Deployment.Delete should have comment or be unexported (golint)
    • Line 215: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 223: warning: exported method Deployment.DeleteByAppID should have comment or be unexported (golint)
    • Line 223: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 232: warning: exported method Deployment.DeleteAllHistory should have comment or be unexported (golint)
    • Line 232: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 241: warning: exported method Deployment.DeleteAllVersion should have comment or be unexported (golint)
    • Line 241: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 250: warning: exported method Deployment.DeleteAllPackage should have comment or be unexported (golint)
    • Line 250: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 261: warning: exported function FindDeploymentsByAppIDs should have comment or be unexported (golint)
    • Line 267: warning: exported function FindDeploymentDetailsByAppID should have comment or be unexported (golint)
    • Line 285: warning: exported type DeploymentHistory should have comment or be unexported (golint)
    • Line 293: warning: exported method DeploymentHistory.Update should have comment or be unexported (golint)
    • Line 293: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 297: warning: exported method DeploymentHistory.Exist should have comment or be unexported (golint)
    • Line 297: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 302: warning: exported method DeploymentHistory.Create should have comment or be unexported (golint)
    • Line 302: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 306: warning: exported method DeploymentHistory.Get should have comment or be unexported (golint)
    • Line 306: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 311: warning: exported method DeploymentHistory.Delete should have comment or be unexported (golint)
    • Line 311: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 320: warning: exported type DeploymentVersion should have comment or be unexported (golint)
    • Line 330: warning: exported method DeploymentVersion.Update should have comment or be unexported (golint)
    • Line 330: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 334: warning: exported method DeploymentVersion.Exist should have comment or be unexported (golint)
    • Line 334: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 339: warning: exported method DeploymentVersion.Create should have comment or be unexported (golint)
    • Line 339: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 343: warning: exported method DeploymentVersion.Get should have comment or be unexported (golint)
    • Line 343: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 348: warning: exported method DeploymentVersion.Delete should have comment or be unexported (golint)
    • Line 348: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • goby/module/context/html_context.go
    • Line 21: warning: exported type HTMLContext should have comment or be unexported (golint)
    • Line 28: warning: exported method HTMLContext.GetErrMsg should have comment or be unexported (golint)
    • Line 32: warning: exported method HTMLContext.HasError should have comment or be unexported (golint)
    • Line 42: warning: exported method HTMLContext.HTML should have comment or be unexported (golint)
    • Line 46: warning: exported method HTMLContext.RenderWithErr should have comment or be unexported (golint)
    • Line 55: warning: exported method HTMLContext.Handle should have comment or be unexported (golint)
    • Line 72: warning: exported method HTMLContext.Auth should have comment or be unexported (golint)
    • Line 104: warning: exported function HTMLContexter should have comment or be unexported (golint)
    • goby/module/form/user_form.go
    • Line 8: warning: exported type InstallForm should have comment or be unexported (golint)
    • Line 52: warning: exported method InstallForm.Validate should have comment or be unexported (golint)
    • Line 56: warning: exported type SignUpForm should have comment or be unexported (golint)
    • Line 63: warning: exported method SignUpForm.Validate should have comment or be unexported (golint)
    • Line 67: warning: exported type SignInForm should have comment or be unexported (golint)
    • Line 73: warning: exported method SignInForm.Validate should have comment or be unexported (golint)
    • Line 77: warning: exported type ChangePasswordForm should have comment or be unexported (golint)
    • Line 83: warning: exported method ChangePasswordForm.Validate should have comment or be unexported (golint)
    • Line 87: warning: exported type EmailForm should have comment or be unexported (golint)
    • Line 91: warning: exported method EmailForm.Validate should have comment or be unexported (golint)
    • Line 95: warning: exported type PwdForm should have comment or be unexported (golint)
    • Line 100: warning: exported method PwdForm.Validate should have comment or be unexported (golint)
    • goby/core/userService/user_service.go
    • Line 1: warning: don't use MixedCaps in package name; userService should be userservice (golint)
    • Line 11: warning: should not use dot imports (golint)
    • Line 13: warning: should not use dot imports (golint)
    • Line 47: warning: exported function VerifyActiveCode should have comment or be unexported (golint)
    • Line 84: warning: exported function ChangePwd should have comment or be unexported (golint)
    • Line 88: warning: exported function ResetPwd should have comment or be unexported (golint)
    • Line 116: warning: exported function Active should have comment or be unexported (golint)
    • Line 135: warning: exported function Create should have comment or be unexported (golint)
    • Line 166: warning: exported function GetByID should have comment or be unexported (golint)
    • Line 179: warning: exported function GetByEmail should have comment or be unexported (golint)
    • Line 192: warning: exported function GetByRands should have comment or be unexported (golint)
    • Line 204: warning: exported function SignInWithUserName should have comment or be unexported (golint)
    • Line 221: warning: exported function SignIn should have comment or be unexported (golint)
    • Line 252: warning: exported function QueryUsers should have comment or be unexported (golint)
    • Line 287: warning: exported function UpdateRole should have comment or be unexported (golint)
    • Line 302: warning: exported function UpdateStatus should have comment or be unexported (golint)
    • goby/module/process/manager.go
    • Line 15: warning: exported var ErrExecTimeout should have comment or be unexported (golint)
    • Line 51: warning: comment on exported function ExecDir should be of the form "ExecDir ..." (golint)
    • Line 89: warning: comment on exported function ExecTimeout should be of the form "ExecTimeout ..." (golint)
    • goby/module/form/app_form.go
    • Line 8: warning: exported type AppAddForm should have comment or be unexported (golint)
    • Line 13: warning: exported method AppAddForm.Validate should have comment or be unexported (golint)
    • goby/router/client/client.go
    • Line 9: warning: exported function UpdateGet should have comment or be unexported (golint)
    • Line 29: warning: exported function Download should have comment or be unexported (golint)
    • Line 44: warning: exported function Deploy should have comment or be unexported (golint)
    • goby/module/storage/local.go
    • Line 12: warning: exported function NewLocal should have comment or be unexported (golint)
    • Line 19: warning: exported type Local should have comment or be unexported (golint)
    • Line 23: warning: exported method Local.Upload should have comment or be unexported (golint)
    • Line 23: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 46: warning: exported method Local.Download should have comment or be unexported (golint)
    • Line 46: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 57: warning: exported method Local.GenerateDownloadURL should have comment or be unexported (golint)
    • Line 57: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • goby/module/storage/oss.go
    • Line 14: warning: exported function NewOSS should have comment or be unexported (golint)
    • Line 43: warning: exported type OSS should have comment or be unexported (golint)
    • Line 48: warning: exported method OSS.Upload should have comment or be unexported (golint)
    • Line 48: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 62: warning: exported method OSS.Download should have comment or be unexported (golint)
    • Line 62: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 74: warning: exported method OSS.GenerateDownloadURL should have comment or be unexported (golint)
    • Line 74: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • goby/module/mailer/mailer.go
    • Line 18: warning: exported type Mailer should have comment or be unexported (golint)
    • Line 33: warning: exported type Message should have comment or be unexported (golint)
    • Line 42: warning: exported function NewMessageFrom should have comment or be unexported (golint)
    • Line 67: warning: exported function NewMessage should have comment or be unexported (golint)
    • Line 75: warning: exported function LoginAuth should have comment or be unexported (golint)
    • Line 97: warning: exported type Sender should have comment or be unexported (golint)
    • Line 100: warning: exported method Sender.Send should have comment or be unexported (golint)
    • Line 229: warning: exported function SendAsync should have comment or be unexported (golint)
    • goby/core/core.go
    • Line 10: warning: exported function WrapIntentError should have comment or be unexported (golint)
    • Line 14: warning: exported function OwnerCan should have comment or be unexported (golint)
    • Line 31: warning: exported function OwnerOf should have comment or be unexported (golint)
    • Line 42: warning: exported function CollaboratorOf should have comment or be unexported (golint)
    • Line 56: warning: exported const APP_SUFFIX_IOS should have comment (or a comment on this block) or be unexported (golint)
    • Line 83: warning: exported var ErrUserNameOrPasswordInvalide should have comment or be unexported (golint)
    • goby/model/app.go
    • Line 7: warning: exported type App should have comment or be unexported (golint)
    • Line 16: warning: exported method App.Deployments should have comment or be unexported (golint)
    • Line 16: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 22: warning: exported method App.Collaborators should have comment or be unexported (golint)
    • Line 22: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 28: warning: exported method App.Update should have comment or be unexported (golint)
    • Line 28: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 32: warning: exported method App.Exist should have comment or be unexported (golint)
    • Line 32: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 37: warning: exported method App.Create should have comment or be unexported (golint)
    • Line 37: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 41: warning: exported method App.Get should have comment or be unexported (golint)
    • Line 41: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 45: warning: exported method App.Delete should have comment or be unexported (golint)
    • Line 45: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 53: warning: exported function FindAppsByIDs should have comment or be unexported (golint)
    • goby/router/web/app.go
    • Line 13: warning: should not use dot imports (golint)
    • Line 17: warning: exported const APPS should have comment (or a comment on this block) or be unexported (golint)
    • Line 22: warning: comment on exported function AppsGet should be of the form "AppsGet ..." (golint)
    • Line 35: warning: comment on exported function AppCollaboratorsGet should be of the form "AppCollaboratorsGet ..." (golint)
    • Line 56: warning: comment on exported function AppDeploymentsGet should be of the form "AppDeploymentsGet ..." (golint)
    • Line 107: warning: comment on exported function AppAdd should be of the form "AppAdd ..." (golint)
    • goby/module/context/context.go
    • Line 38: warning: exported function InitContextConfig should have comment or be unexported (golint)
    • Line 49: warning: exported type Context should have comment or be unexported (golint)
    • Line 58: warning: exported method Context.Query should have comment or be unexported (golint)
    • Line 63: warning: exported method Context.ServeFile should have comment or be unexported (golint)
    • Line 80: warning: exported method Context.ServeContent should have comment or be unexported (golint)
    • Line 98: warning: exported method Context.Handle should have comment or be unexported (golint)
    • Line 115: warning: exported method Context.Auth should have comment or be unexported (golint)
    • Line 119: warning: exported function Contexter should have comment or be unexported (golint)
    • goby/model/user_token.go
    • Line 9: warning: exported type UserToken should have comment or be unexported (golint)
    • Line 22: warning: exported method UserToken.Update should have comment or be unexported (golint)
    • Line 22: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 26: warning: exported method UserToken.Exist should have comment or be unexported (golint)
    • Line 26: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 31: warning: exported method UserToken.Create should have comment or be unexported (golint)
    • Line 31: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 35: warning: exported method UserToken.Get should have comment or be unexported (golint)
    • Line 35: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 40: warning: exported method UserToken.GetUnexpired should have comment or be unexported (golint)
    • Line 40: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 52: warning: exported method UserToken.Delete should have comment or be unexported (golint)
    • Line 52: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 61: warning: exported method UserToken.DeleteByNameAndUID should have comment or be unexported (golint)
    • Line 61: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 69: warning: exported method UserToken.DeleteByCreatorAndUID should have comment or be unexported (golint)
    • Line 69: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 77: warning: exported method UserToken.Convert should have comment or be unexported (golint)
    • Line 77: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • goby/router/accessKey/access_key.go
    • Line 1: warning: don't use MixedCaps in package name; accessKey should be accesskey (golint)
    • Line 8: warning: exported function Add should have comment or be unexported (golint)
    • Line 25: warning: exported function Get should have comment or be unexported (golint)
    • Line 41: warning: exported function Delete should have comment or be unexported (golint)
    • Line 54: warning: exported function Patch should have comment or be unexported (golint)
    • goby/cmd/web.go
    • Line 15: warning: should not use dot imports (golint)
    • Line 38: warning: exported var CmdWeb should have comment or be unexported (golint)
    • goby/core/clientService/client_service.go
    • Line 1: warning: don't use MixedCaps in package name; clientService should be clientservice (golint)
    • Line 10: warning: should not use dot imports (golint)
    • Line 18: warning: exported function UpdateCheck should have comment or be unexported (golint)
    • Line 90: warning: exported function ReportStatusDownload should have comment or be unexported (golint)
    • Line 112: warning: exported function ReportStatusDeploy should have comment or be unexported (golint)
    • goby/goby.go
    • Line 13: warning: exported const APP_VER should have comment or be unexported (golint)
    • goby/router/app/deployment/deployment.go
    • Line 11: warning: exported function Post should have comment or be unexported (golint)
    • Line 25: warning: exported function Patch should have comment or be unexported (golint)
    • Line 36: warning: exported function Delete should have comment or be unexported (golint)
    • Line 46: warning: exported function Get should have comment or be unexported (golint)
    • Line 61: warning: exported function MetricsGet should have comment or be unexported (golint)
    • Line 75: warning: exported function HistoryGet should have comment or be unexported (golint)
    • Line 88: warning: exported function HistoryDelete should have comment or be unexported (golint)
    • Line 99: warning: exported function ReleasePost should have comment or be unexported (golint)
    • Line 127: warning: exported function ReleasePatch should have comment or be unexported (golint)
    • Line 137: warning: exported function RobackPost should have comment or be unexported (golint)
    • Line 148: warning: exported function RobackForLabelPost should have comment or be unexported (golint)
    • Line 160: warning: exported function DestPost should have comment or be unexported (golint)
    • goby/module/storage/qiniu.go
    • Line 13: warning: exported function NewQiNiu should have comment or be unexported (golint)
    • Line 25: warning: exported type QiNiu should have comment or be unexported (golint)
    • Line 30: warning: exported method QiNiu.Upload should have comment or be unexported (golint)
    • Line 30: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 40: warning: exported method QiNiu.Download should have comment or be unexported (golint)
    • Line 40: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 53: warning: exported method QiNiu.GenerateDownloadURL should have comment or be unexported (golint)
    • Line 53: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • goby/module/storage/storage.go
    • Line 3: warning: exported type Config should have comment or be unexported (golint)
    • Line 18: warning: exported type Storage should have comment or be unexported (golint)
    • Line 30: warning: exported function InitStorage should have comment or be unexported (golint)
    • Line 42: warning: exported function Upload should have comment or be unexported (golint)
    • Line 46: warning: exported function Download should have comment or be unexported (golint)
    • Line 50: warning: exported function GenerateDownloadURL should have comment or be unexported (golint)
    • goby/model/collaborator.go
    • Line 7: warning: exported type Collaborator should have comment or be unexported (golint)
    • Line 18: warning: exported method Collaborator.App should have comment or be unexported (golint)
    • Line 18: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 30: warning: exported method Collaborator.Update should have comment or be unexported (golint)
    • Line 30: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 34: warning: exported method Collaborator.Exist should have comment or be unexported (golint)
    • Line 34: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 39: warning: exported method Collaborator.Create should have comment or be unexported (golint)
    • Line 39: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 43: warning: exported method Collaborator.Get should have comment or be unexported (golint)
    • Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 47: warning: exported method Collaborator.Delete should have comment or be unexported (golint)
    • Line 47: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 55: warning: exported method Collaborator.DeleteByAppID should have comment or be unexported (golint)
    • Line 55: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 64: warning: exported method Collaborator.DeleteByAppIDAndUID should have comment or be unexported (golint)
    • Line 64: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 73: warning: exported function FindCollaboratorsByAppIDs should have comment or be unexported (golint)
    • Line 78: warning: exported function FindOwnerByAppName should have comment or be unexported (golint)
    • Line 96: warning: exported function FindCollaboratorByAppNameAndUID should have comment or be unexported (golint)
    • goby/module/context/api_context.go
    • Line 17: warning: exported type APIContext should have comment or be unexported (golint)
    • Line 50: warning: exported method APIContext.JSON should have comment or be unexported (golint)
    • Line 59: warning: exported method APIContext.Auth should have comment or be unexported (golint)
    • Line 77: warning: exported method APIContext.GetToken should have comment or be unexported (golint)
    • Line 118: warning: exported function APIContexter should have comment or be unexported (golint)
    • goby/router/app/app.go
    • Line 9: warning: exported function Get should have comment or be unexported (golint)
    • Line 21: warning: exported function Post should have comment or be unexported (golint)
    • Line 34: warning: exported function Delete should have comment or be unexported (golint)
    • Line 47: warning: exported function Patch should have comment or be unexported (golint)
    • Line 60: warning: exported function TransferPost should have comment or be unexported (golint)
    • goby/module/setting/setting.go
    • Line 24: warning: exported type Scheme should have comment or be unexported (golint)
    • Line 27: warning: exported const SCHEME_HTTP should have comment (or a comment on this block) or be unexported (golint)
    • Line 32: warning: comment on exported var AppVer should be of the form "AppVer ..." (golint)
    • Line 34: warning: exported var AppName should have comment or be unexported (golint)
    • Line 40: warning: comment on exported var Protocol should be of the form "Protocol ..." (golint)
    • Line 43: warning: exported var HTTPPort should have its own declaration (golint)
    • Line 48: warning: exported var KeyFile should have its own declaration (golint)
    • Line 52: warning: comment on exported var AttachmentPath should be of the form "AttachmentPath ..." (golint)
    • Line 59: warning: comment on exported var UseSQLite3 should be of the form "UseSQLite3 ..." (golint)
    • Line 66: warning: comment on exported var InstallLock should be of the form "InstallLock ..." (golint)
    • Line 75: warning: comment on exported var LogRootPath should be of the form "LogRootPath ..." (golint)
    • Line 80: warning: comment on exported var TimeFormat should be of the form "TimeFormat ..." (golint)
    • Line 83: warning: comment on exported var CacheAdapter should be of the form "CacheAdapter ..." (golint)
    • Line 93: warning: comment on exported var SessionConfig should be of the form "SessionConfig ..." (golint)
    • Line 97: warning: comment on exported var Cfg should be of the form "Cfg ..." (golint)
    • Line 108: warning: exported function ExecPath should have comment or be unexported (golint)
    • Line 120: warning: exported function WorkDir should have comment or be unexported (golint)
    • Line 125: warning: exported function InitConfig should have comment or be unexported (golint)
    • Line 230: warning: exported var Service should have comment or be unexported (golint)
    • Line 348: warning: exported var DBConfig should have comment or be unexported (golint)
    • Line 401: warning: exported var Storage should have comment or be unexported (golint)
    • Line 423: warning: exported var MailService should have comment or be unexported (golint)
    • Line 459: warning: exported var PackageConfig should have comment or be unexported (golint)
    • goby/module/form/api_form.go
    • Line 3: warning: exported type AppOrDeploymentOption should have comment or be unexported (golint)
    • Line 7: warning: exported type PackageInfo should have comment or be unexported (golint)
    • Line 16: warning: exported type UpdatePackageInfo should have comment or be unexported (golint)
    • Line 20: warning: exported type ReportStatus should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign96%

IneffAssign detects ineffectual assignments in Go code.


misspell87%

Misspell Finds commonly misspelled English words