Go Report Card

Preparing report...

Report for github.com/driusan/bug

A    Great!    Found 38 issues across 45 files

Tweet
Last refresh:

Refresh now

gofmt86%

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!

gocyclo93%

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.

    • bug/main.go
    • Line 11: warning: cyclomatic complexity 30 of function main() is high (> 15) (gocyclo)

golint26%

Golint is a linter for Go source code.

    • bug/bugapp/List.go
    • Line 13: 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 19: warning: should omit 2nd value from range; this loop is equivalent to `for idx := range ...` (golint)
    • Line 29: warning: exported function List should have comment or be unexported (golint)
    • bug/bugapp/utils.go
    • Line 3: warning: exported type ArgumentList should have comment or be unexported (golint)
    • Line 5: warning: exported method ArgumentList.HasArgument should have comment or be unexported (golint)
    • Line 14: warning: exported method ArgumentList.GetArgument should have comment or be unexported (golint)
    • Line 30: warning: exported method ArgumentList.GetAndRemoveArguments should have comment or be unexported (golint)
    • bug/scm/Interfaces.go
    • Line 5: warning: exported type SCMHandler should have comment or be unexported (golint)
    • Line 11: warning: exported type FileStatus should have comment or be unexported (golint)
    • bug/scm/GitManager.go
    • Line 14: warning: exported type GitManager should have comment or be unexported (golint)
    • Line 19: warning: exported method GitManager.Purge should have comment or be unexported (golint)
    • Line 162: warning: exported method GitManager.Commit should have comment or be unexported (golint)
    • Line 199: warning: exported method GitManager.GetSCMType should have comment or be unexported (golint)
    • bug/scm/HgManager.go
    • Line 9: warning: exported type HgManager should have comment or be unexported (golint)
    • Line 11: warning: exported method HgManager.Purge should have comment or be unexported (golint)
    • Line 15: warning: exported method HgManager.Commit should have comment or be unexported (golint)
    • Line 30: warning: exported method HgManager.GetSCMType should have comment or be unexported (golint)
    • bug/bugapp/Roadmap.go
    • Line 12: warning: exported type BugListByMilestone should have comment or be unexported (golint)
    • Line 47: warning: exported function Roadmap should have comment or be unexported (golint)
    • bug/bugs/Bug.go
    • Line 12: warning: error var NoDescriptionError should have name of the form ErrFoo (golint)
    • Line 12: warning: exported var NoDescriptionError should have comment or be unexported (golint)
    • Line 13: warning: error var NotFoundError should have name of the form ErrFoo (golint)
    • Line 13: warning: exported var NotFoundError should have comment or be unexported (golint)
    • Line 15: warning: exported type Bug should have comment or be unexported (golint)
    • Line 20: warning: exported type Tag should have comment or be unexported (golint)
    • Line 22: warning: exported function TitleToDir should have comment or be unexported (golint)
    • Line 51: warning: exported method Bug.GetDirectory should have comment or be unexported (golint)
    • Line 55: warning: exported method Bug.LoadBug should have comment or be unexported (golint)
    • Line 60: warning: exported method Bug.Title should have comment or be unexported (golint)
    • Line 94: warning: exported method Bug.Description should have comment or be unexported (golint)
    • Line 109: warning: exported method Bug.SetDescription should have comment or be unexported (golint)
    • Line 114: warning: exported method Bug.RemoveTag should have comment or be unexported (golint)
    • Line 121: warning: exported method Bug.TagBug should have comment or be unexported (golint)
    • Line 129: warning: exported method Bug.ViewBug should have comment or be unexported (golint)
    • Line 152: warning: exported method Bug.StringTags should have comment or be unexported (golint)
    • Line 167: warning: exported method Bug.HasTag should have comment or be unexported (golint)
    • Line 176: warning: exported method Bug.Tags should have comment or be unexported (golint)
    • Line 229: warning: exported method Bug.Status should have comment or be unexported (golint)
    • Line 233: warning: exported method Bug.SetStatus should have comment or be unexported (golint)
    • Line 236: warning: exported method Bug.Priority should have comment or be unexported (golint)
    • Line 240: warning: exported method Bug.SetPriority should have comment or be unexported (golint)
    • Line 243: warning: exported method Bug.Milestone should have comment or be unexported (golint)
    • Line 247: warning: exported method Bug.SetMilestone should have comment or be unexported (golint)
    • Line 251: warning: exported method Bug.Identifier should have comment or be unexported (golint)
    • Line 255: warning: exported method Bug.SetIdentifier should have comment or be unexported (golint)
    • Line 259: warning: exported function New should have comment or be unexported (golint)
    • bug/scm/Detect.go
    • Line 9: warning: exported type SCMNotFound should have comment or be unexported (golint)
    • Line 35: warning: exported function DetectSCM should have comment or be unexported (golint)
    • bug/bugs/IO.go
    • Line 36: warning: exported method Bug.WriteAt should have comment or be unexported (golint)
    • Line 49: warning: exported method Bug.Close should have comment or be unexported (golint)
    • Line 58: warning: exported method Bug.Remove should have comment or be unexported (golint)
    • bug/scm/GitManager_test.go
    • Line 59: warning: receiver name g should be consistent with previous receiver name t for GitTester (golint)
    • Line 73: warning: receiver name g should be consistent with previous receiver name t for GitTester (golint)
    • Line 100: warning: receiver name m should be consistent with previous receiver name t for GitTester (golint)
    • Line 110: warning: receiver name m should be consistent with previous receiver name t for GitTester (golint)
    • bug/bugs/Directory.go
    • Line 10: warning: exported function GetRootDir should have comment or be unexported (golint)
    • Line 35: warning: exported function GetIssuesDir should have comment or be unexported (golint)
    • Line 43: warning: exported type Directory should have comment or be unexported (golint)
    • Line 45: warning: exported method Directory.GetShortName should have comment or be unexported (golint)
    • Line 50: warning: exported method Directory.ToTitle should have comment or be unexported (golint)
    • Line 64: warning: exported method Directory.LastModified should have comment or be unexported (golint)
    • bug/bugs/Find.go
    • Line 10: warning: exported type BugNotFoundError should have comment or be unexported (golint)
    • Line 15: warning: exported function FindBugsByTag should have comment or be unexported (golint)
    • Line 35: warning: exported function LoadBugByDirectory should have comment or be unexported (golint)
    • Line 44: warning: exported function LoadBugByHeuristic should have comment or be unexported (golint)
    • Line 70: warning: exported function LoadBugByStringIndex should have comment or be unexported (golint)
    • Line 87: warning: exported function LoadBugByIdentifier should have comment or be unexported (golint)
    • Line 101: warning: exported function LoadBugByIndex should have comment or be unexported (golint)
    • Line 113: warning: exported function GetAllBugs should have comment or be unexported (golint)
    • bug/bug-import/utils.go
    • Line 3: warning: exported type ArgumentList should have comment or be unexported (golint)
    • Line 5: warning: exported method ArgumentList.HasArgument should have comment or be unexported (golint)
    • Line 14: warning: exported method ArgumentList.GetArgument should have comment or be unexported (golint)
    • Line 30: warning: exported method ArgumentList.GetAndRemoveArguments should have comment or be unexported (golint)
    • bug/scm/HgManager_test.go
    • Line 52: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 68: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 72: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 88: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 92: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 95: warning: receiver name c should be consistent with previous receiver name t for HgTester (golint)
    • Line 106: warning: receiver name m should be consistent with previous receiver name t for HgTester (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign93%

IneffAssign detects ineffectual assignments in Go code.

misspell95%

Misspell Finds commonly misspelled English words

    • bug/scm/GitManager.go
    • Line 165: warning: "commited" is a misspelling of "committed" (misspell)
    • Line 174: warning: "commited" is a misspelling of "committed" (misspell)
    • Line 193: warning: "commited" is a misspelling of "committed" (misspell)