Preparing report...

Report for github.com/golobby/orm

A+    Excellent!    Found 9 issues across 15 files

Tweet

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!


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!


golint46%

Golint is a linter for Go source code.

    • orm/delete.go
    • Line 10: warning: exported type DeleteStmt should have comment or be unexported (golint)
    • Line 15: warning: exported method DeleteStmt.Where should have comment or be unexported (golint)
    • Line 24: warning: exported method DeleteStmt.OrWhere should have comment or be unexported (golint)
    • Line 29: warning: exported method DeleteStmt.AndWhere should have comment or be unexported (golint)
    • Line 33: warning: exported method DeleteStmt.SQL should have comment or be unexported (golint)
    • Line 33: warning: receiver name d should be consistent with previous receiver name q for DeleteStmt (golint)
    • Line 37: warning: exported method DeleteStmt.ExecContext should have comment or be unexported (golint)
    • Line 37: warning: receiver name d should be consistent with previous receiver name q for DeleteStmt (golint)
    • Line 44: warning: exported method DeleteStmt.Exec should have comment or be unexported (golint)
    • Line 44: warning: receiver name d should be consistent with previous receiver name q for DeleteStmt (golint)
    • Line 53: warning: exported function NewDelete should have comment or be unexported (golint)
    • orm/helpers.go
    • Line 15: warning: exported var PlaceHolderGenerators should have comment or be unexported (golint)
    • orm/insert.go
    • Line 10: warning: exported type InsertStmt should have comment or be unexported (golint)
    • Line 17: warning: exported function NewInsert should have comment or be unexported (golint)
    • Line 21: warning: exported method InsertStmt.Into should have comment or be unexported (golint)
    • Line 26: warning: exported method InsertStmt.Values should have comment or be unexported (golint)
    • Line 31: warning: exported method InsertStmt.PlaceHolderGenerator should have comment or be unexported (golint)
    • Line 36: warning: exported method InsertStmt.Exec should have comment or be unexported (golint)
    • Line 43: warning: exported method InsertStmt.ExecContext should have comment or be unexported (golint)
    • Line 51: warning: exported method InsertStmt.SQL should have comment or be unexported (golint)
    • orm/obj.go
    • Line 128: warning: exported type PKValue should have comment or be unexported (golint)
    • Line 154: warning: exported type SetPK should have comment or be unexported (golint)
    • Line 181: warning: exported type ObjectMetadata should have comment or be unexported (golint)
    • Line 193: warning: exported function ObjectMetadataFrom should have comment or be unexported (golint)
    • orm/select.go
    • Line 10: warning: exported type ClauseType should have comment or be unexported (golint)
    • Line 13: warning: don't use underscores in Go names; const ClauseType_Where should be ClauseTypeWhere (golint)
    • Line 13: warning: exported const ClauseType_Where should have comment (or a comment on this block) or be unexported (golint)
    • Line 14: warning: don't use underscores in Go names; const ClauseType_Limit should be ClauseTypeLimit (golint)
    • Line 15: warning: don't use underscores in Go names; const ClauseType_Offset should be ClauseTypeOffset (golint)
    • Line 16: warning: don't use underscores in Go names; const ClauseType_OrderBy should be ClauseTypeOrderBy (golint)
    • Line 17: warning: don't use underscores in Go names; const ClauseType_GroupBy should be ClauseTypeGroupBy (golint)
    • Line 18: warning: don't use underscores in Go names; const ClauseType_InnerJoin should be ClauseTypeInnerJoin (golint)
    • Line 19: warning: don't use underscores in Go names; const ClauseType_LeftJoin should be ClauseTypeLeftJoin (golint)
    • Line 20: warning: don't use underscores in Go names; const ClauseType_RightJoin should be ClauseTypeRightJoin (golint)
    • Line 21: warning: don't use underscores in Go names; const ClauseType_FullOuterJoin should be ClauseTypeFullOuterJoin (golint)
    • Line 22: warning: don't use underscores in Go names; const ClauseType_Select should be ClauseTypeSelect (golint)
    • Line 23: warning: don't use underscores in Go names; const ClauseType_Having should be ClauseTypeHaving (golint)
    • Line 26: warning: exported type Clause should have comment or be unexported (golint)
    • Line 39: warning: exported type Schema should have comment or be unexported (golint)
    • Line 44: warning: exported function NewSchema should have comment or be unexported (golint)
    • Line 57: warning: exported method Schema.NewModel should have comment or be unexported (golint)
    • Line 96: warning: exported type SelectStmt should have comment or be unexported (golint)
    • Line 109: warning: exported method SelectStmt.Schema should have comment or be unexported (golint)
    • Line 115: warning: exported method SelectStmt.Having should have comment or be unexported (golint)
    • Line 115: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 129: warning: exported method SelectStmt.Limit should have comment or be unexported (golint)
    • Line 129: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 134: warning: exported method SelectStmt.Offset should have comment or be unexported (golint)
    • Line 134: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 139: warning: exported method SelectStmt.Skip should have comment or be unexported (golint)
    • Line 139: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 143: warning: exported method SelectStmt.Take should have comment or be unexported (golint)
    • Line 143: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 147: warning: exported method SelectStmt.InnerJoin should have comment or be unexported (golint)
    • Line 147: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 155: warning: exported method SelectStmt.RightJoin should have comment or be unexported (golint)
    • Line 155: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 162: warning: exported method SelectStmt.LeftJoin should have comment or be unexported (golint)
    • Line 162: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 169: warning: exported method SelectStmt.FullOuterJoin should have comment or be unexported (golint)
    • Line 169: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 178: warning: exported method SelectStmt.OrderBy should have comment or be unexported (golint)
    • Line 178: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 191: warning: exported method SelectStmt.Select should have comment or be unexported (golint)
    • Line 191: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 203: warning: exported method SelectStmt.Distinct should have comment or be unexported (golint)
    • Line 208: warning: exported method SelectStmt.Table should have comment or be unexported (golint)
    • Line 208: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 213: warning: exported method SelectStmt.GroupBy should have comment or be unexported (golint)
    • Line 213: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 227: warning: exported method SelectStmt.Where should have comment or be unexported (golint)
    • Line 227: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 241: warning: exported method SelectStmt.OrWhere should have comment or be unexported (golint)
    • Line 241: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 251: warning: exported method SelectStmt.AndWhere should have comment or be unexported (golint)
    • Line 251: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 255: warning: exported method SelectStmt.SQL should have comment or be unexported (golint)
    • Line 255: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 305: warning: exported method SelectStmt.Exec should have comment or be unexported (golint)
    • Line 305: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 314: warning: exported method SelectStmt.ExecContext should have comment or be unexported (golint)
    • Line 314: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 322: warning: exported method SelectStmt.Bind should have comment or be unexported (golint)
    • Line 322: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 330: warning: exported method SelectStmt.BindContext should have comment or be unexported (golint)
    • Line 330: warning: receiver name q should be consistent with previous receiver name s for SelectStmt (golint)
    • Line 338: warning: exported function NewQuery should have comment or be unexported (golint)
    • orm/update.go
    • Line 10: warning: exported type UpdateStmt should have comment or be unexported (golint)
    • Line 16: warning: exported method UpdateStmt.Where should have comment or be unexported (golint)
    • Line 25: warning: exported method UpdateStmt.OrWhere should have comment or be unexported (golint)
    • Line 30: warning: exported method UpdateStmt.AndWhere should have comment or be unexported (golint)
    • Line 34: warning: exported type KV should have comment or be unexported (golint)
    • Line 36: warning: exported method UpdateStmt.Set should have comment or be unexported (golint)
    • Line 36: warning: receiver name u should be consistent with previous receiver name q for UpdateStmt (golint)
    • Line 46: warning: exported method UpdateStmt.SQL should have comment or be unexported (golint)
    • Line 46: warning: receiver name u should be consistent with previous receiver name q for UpdateStmt (golint)
    • Line 50: warning: exported method UpdateStmt.ExecContext should have comment or be unexported (golint)
    • Line 50: warning: receiver name d should be consistent with previous receiver name q for UpdateStmt (golint)
    • Line 57: warning: exported method UpdateStmt.Exec should have comment or be unexported (golint)
    • Line 57: warning: receiver name d should be consistent with previous receiver name q for UpdateStmt (golint)
    • Line 65: warning: exported function NewUpdate should have comment or be unexported (golint)
    • orm/where.go
    • Line 21: warning: exported var WhereHelpers should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign93%

IneffAssign detects ineffectual assignments in Go code.


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!