Report for github.com/cube2222/jql
A Great! Found 8 issues across 9 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo88%
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.
-
-
jql/jql/functions/functions.go
- Line 56: warning: cyclomatic complexity 16 of function GetElement() is high (> 15) (gocyclo)
golint33%
Golint is a linter for Go source code.
-
-
jql/jql/jql.go
- Line 3: warning: exported type Expression should have comment or be unexported (golint)
- Line 7: warning: exported type Constant should have comment or be unexported (golint)
- Line 11: warning: exported function NewConstant should have comment or be unexported (golint)
- Line 15: warning: exported method Constant.Get should have comment or be unexported (golint)
-
-
jql/jql/parser/ast.go
- Line 9: warning: exported type Query should have comment or be unexported (golint)
- Line 13: warning: exported type ExpressionConstructorContext should have comment or be unexported (golint)
- Line 18: warning: exported type Expression should have comment or be unexported (golint)
- Line 23: warning: exported type SExpression should have comment or be unexported (golint)
- Line 28: warning: exported method SExpression.GetExecutionExpression should have comment or be unexported (golint)
- Line 52: warning: exported method SExpression.IExpression should have comment or be unexported (golint)
- Line 54: warning: exported type Expressions should have comment or be unexported (golint)
- Line 56: warning: exported type Constant should have comment or be unexported (golint)
- Line 60: warning: exported method Constant.IExpression should have comment or be unexported (golint)
- Line 62: warning: exported method Constant.GetExecutionExpression should have comment or be unexported (golint)
-
-
jql/jql/parser/parser.go
- Line 3: warning: exported function Parse should have comment or be unexported (golint)
-
-
jql/jql/parser/token.go
- Line 11: warning: exported type Tokenizer should have comment or be unexported (golint)
- Line 25: warning: exported method Tokenizer.Lex should have comment or be unexported (golint)
-
-
jql/jql/app/app.go
- Line 12: warning: exported type Input should have comment or be unexported (golint)
- Line 16: warning: exported type Output should have comment or be unexported (golint)
- Line 20: warning: exported type App should have comment or be unexported (golint)
- Line 26: warning: exported function NewApp should have comment or be unexported (golint)
- Line 34: warning: exported method App.Run should have comment or be unexported (golint)
-
-
jql/jql/functions/functions.go
- Line 12: warning: exported var Functions should have comment or be unexported (golint)
- Line 34: warning: exported type Element should have comment or be unexported (golint)
- Line 39: warning: exported function NewElement should have comment or be unexported (golint)
- Line 56: warning: exported function GetElement should have comment or be unexported (golint)
- Line 120: warning: exported method Element.Get should have comment or be unexported (golint)
- Line 129: warning: exported type Keys should have comment or be unexported (golint)
- Line 132: warning: exported function NewKeys should have comment or be unexported (golint)
- Line 140: warning: exported method Keys.Get should have comment or be unexported (golint)
- Line 163: warning: exported type Identity should have comment or be unexported (golint)
- Line 166: warning: exported function NewIdentity should have comment or be unexported (golint)
- Line 174: warning: exported method Identity.Get should have comment or be unexported (golint)
- Line 178: warning: exported type Array should have comment or be unexported (golint)
- Line 182: warning: exported function NewArray should have comment or be unexported (golint)
- Line 186: warning: exported method Array.Get should have comment or be unexported (golint)
- Line 199: warning: exported type Object should have comment or be unexported (golint)
- Line 204: warning: exported function NewObject should have comment or be unexported (golint)
- Line 220: warning: exported method Object.Get should have comment or be unexported (golint)
- Line 242: warning: exported type Pipe should have comment or be unexported (golint)
- Line 246: warning: exported function NewPipe should have comment or be unexported (golint)
- Line 253: warning: exported method Pipe.Get should have comment or be unexported (golint)
- Line 265: warning: exported type Sprintf should have comment or be unexported (golint)
- Line 270: warning: exported function NewSprintf should have comment or be unexported (golint)
- Line 277: warning: exported method Sprintf.Get should have comment or be unexported (golint)
- Line 298: warning: exported type Join should have comment or be unexported (golint)
- Line 303: warning: exported function NewJoin should have comment or be unexported (golint)
- Line 320: warning: exported method Join.Get should have comment or be unexported (golint)
- Line 347: warning: exported function IsTruthy should have comment or be unexported (golint)
- Line 357: warning: exported type Filter should have comment or be unexported (golint)
- Line 362: warning: exported function NewFilter should have comment or be unexported (golint)
- Line 379: warning: exported method Filter.Get should have comment or be unexported (golint)
- Line 401: warning: exported type Equal should have comment or be unexported (golint)
- Line 406: warning: exported function NewEqual should have comment or be unexported (golint)
- Line 417: warning: exported method Equal.Get should have comment or be unexported (golint)
- Line 430: warning: exported function Floatify should have comment or be unexported (golint)
- Line 441: warning: exported type LessThan should have comment or be unexported (golint)
- Line 446: warning: exported function NewLessThan should have comment or be unexported (golint)
- Line 457: warning: exported method LessThan.Get should have comment or be unexported (golint)
- Line 491: warning: exported type GreaterThan should have comment or be unexported (golint)
- Line 496: warning: exported function NewGreaterThan should have comment or be unexported (golint)
- Line 507: warning: exported method GreaterThan.Get should have comment or be unexported (golint)
- Line 541: warning: exported type Range should have comment or be unexported (golint)
- Line 546: warning: exported function NewRange should have comment or be unexported (golint)
- Line 563: warning: exported method Range.Get should have comment or be unexported (golint)
- Line 590: warning: exported type And should have comment or be unexported (golint)
- Line 594: warning: exported function NewAnd should have comment or be unexported (golint)
- Line 598: warning: exported method And.Get should have comment or be unexported (golint)
- Line 612: warning: exported type Or should have comment or be unexported (golint)
- Line 616: warning: exported function NewOr should have comment or be unexported (golint)
- Line 620: warning: exported method Or.Get should have comment or be unexported (golint)
- Line 634: warning: exported type Not should have comment or be unexported (golint)
- Line 638: warning: exported function NewNot should have comment or be unexported (golint)
- Line 645: warning: exported method Not.Get should have comment or be unexported (golint)
- Line 654: warning: exported type IfTE should have comment or be unexported (golint)
- Line 660: warning: exported function NewIfTE should have comment or be unexported (golint)
- Line 671: warning: exported method IfTE.Get should have comment or be unexported (golint)
- Line 683: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
- Line 692: warning: exported type Error should have comment or be unexported (golint)
- Line 696: warning: exported function NewError should have comment or be unexported (golint)
- Line 705: warning: exported method Error.Get should have comment or be unexported (golint)
- Line 714: warning: exported type Recover should have comment or be unexported (golint)
- Line 718: warning: exported function NewRecover should have comment or be unexported (golint)
- Line 727: warning: exported method Recover.Get should have comment or be unexported (golint)
ineffassign44%
IneffAssign detects ineffectual assignments in Go code.
-
-
/home/shawn/mygo/src/github.com/gojp/goreportcard/_repos/src/github.com/cube2222/jql/jql/parser/token.go
- Line 25: warning: undeclared name: yySymType (ineffassign)
- Line 25: warning: undeclared name: yySymType (ineffassign)
-
-
jql/cmd/root.go
- Line 32: warning: undeclared name: MustAsset (ineffassign)
- Line 32: warning: undeclared name: MustAsset (ineffassign)
-
-
jql/jql/parser/token.go
- Line 25: warning: undeclared name: yySymType (ineffassign)
- Line 39: warning: undeclared name: BOOLEAN (ineffassign)
- Line 44: warning: undeclared name: BOOLEAN (ineffassign)
- Line 49: warning: undeclared name: NULL (ineffassign)
- Line 56: warning: undeclared name: ID (ineffassign)
- Line 67: warning: undeclared name: INTEGER (ineffassign)
- Line 75: warning: undeclared name: STRING (ineffassign)
- Line 25: warning: undeclared name: yySymType (ineffassign)
-
-
jql/jql/parser/parser.go
- Line 8: warning: undeclared name: yyParse (ineffassign)
-
-
jql/jql/functions/functions.go
- Line 731: warning: ineffectual assignment to err (ineffassign)