gofmt72%

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

• picard/examples/product/product_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/examples/blog/blog_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/tags/tags_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

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

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

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

• picard/examples/product/product.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/queryparts/join.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/query/hydrate_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/query/query_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• picard/query/testing.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

go_vet100%

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

No problems detected. Good job!

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.

• picard/filter.go
  • Line 221: warning: cyclomatic complexity 18 of function (PersistenceORM).FilterModel() is high (> 15) (gocyclo)

• picard/query/build.go
  • Line 69: warning: cyclomatic complexity 28 of function buildQuery() is high (> 15) (gocyclo)

• picard/tags/tags.go
  • Line 607: warning: cyclomatic complexity 20 of function TableMetadataFromType() is high (> 15) (gocyclo)

• picard/picard_test.go
  • Line 146: warning: cyclomatic complexity 38 of function TestDeployments() is high (> 15) (gocyclo)

• picard/picard.go
  • Line 930: warning: cyclomatic complexity 30 of function (PersistenceORM).processObject() is high (> 15) (gocyclo)
  • Line 152: warning: cyclomatic complexity 19 of function (PersistenceORM).upsert() is high (> 15) (gocyclo)
  • Line 690: warning: cyclomatic complexity 19 of function (PersistenceORM).performChildUpserts() is high (> 15) (gocyclo)

golint70%

Golint is a linter for Go source code.

• picard/queryparts/fielddescriptor.go
  • Line 14: warning: exported const AliasedField should have comment (or a comment on this block) or be unexported (golint)

• picard/decoding/decoding.go
  • Line 1: warning: package comment should be of the form "Package decoding ..." (golint)

• picard/crypto/crypto.go
  • Line 14: warning: exported function SetEncryptionKey should have comment or be unexported (golint)
  • Line 22: warning: exported function GetEncryptionKey should have comment or be unexported (golint)
  • Line 29: warning: exported function GenerateNewEncryptionKey should have comment or be unexported (golint)
  • Line 38: warning: exported function EncryptBytes should have comment or be unexported (golint)
  • Line 64: warning: exported function DecryptBytes should have comment or be unexported (golint)

• picard/picard_test/mock.go
  • Line 1: warning: package comment should be of the form "Package picard_test ..." (golint)
  • Line 195: warning: comment on exported method MultiMockORM.Rollback should be of the form "Rollback ..." (golint)

• picard/tags/tags.go
  • Line 19: warning: comment on exported type Association should be of the form "Association ..." (with optional leading article) (golint)
  • Line 221: warning: comment on exported type FieldFilter should be of the form "FieldFilter ..." (with optional leading article) (golint)

• picard/query/query.go
  • Line 18: warning: comment on exported function NewAliased should be of the form "NewAliased ..." (golint)

• picard/doc.go
  • Line 1: warning: package comment should be of the form "Package picard ..." (golint)

• picard/picard.go
  • Line 61: warning: comment on exported method PersistenceORM.StartTransaction should be of the form "StartTransaction ..." (golint)

• picard/metadata/metadata.go
  • Line 10: warning: exported type Metadata should have comment or be unexported (golint)
  • Line 14: warning: exported function AddDefinedField should have comment or be unexported (golint)
  • Line 22: warning: exported function HasDefinedFields should have comment or be unexported (golint)
  • Line 30: warning: exported function InitializeDefinedFields should have comment or be unexported (golint)
  • Line 37: warning: exported function GetMetadataValue should have comment or be unexported (golint)
  • Line 50: warning: exported function GetMetadataFromPicardStruct should have comment or be unexported (golint)

• picard/reflectutil/reflect.go
  • Line 1: warning: package comment should not have leading space (golint)

• picard/examples/product/product.go
  • Line 15: warning: exported type Product should have comment or be unexported (golint)
  • Line 29: warning: exported type Order should have comment or be unexported (golint)

• picard/filter.go
  • Line 15: warning: comment on exported type FilterRequest should be of the form "FilterRequest ..." (with optional leading article) (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign95%

IneffAssign detects ineffectual assignments in Go code.

• picard/examples/product/product.go
  • Line 97: warning: ineffectual assignment to err (ineffassign)

• picard/picard_test.go
  • Line 1800: warning: ineffectual assignment to err (ineffassign)
  • Line 1888: warning: ineffectual assignment to err (ineffassign)

misspell95%

Misspell Finds commonly misspelled English words

• picard/examples/blog/blog.go
  • Line 240: warning: "assocation" is a misspelling of "association" (misspell)

• picard/filter.go
  • Line 269: warning: "funcitonality" is a misspelling of "functionality" (misspell)