Go Report Card

Preparing report...

Report for github.com/gregod-com/grgd

A+    Excellent!    Found 21 issues across 77 files

Tweet
Last refresh:

Refresh now

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!

gocyclo94%

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.

    • grgd/core/core.go
    • Line 16: warning: cyclomatic complexity 16 of function RegisterDependecies() is high (> 15) (gocyclo)

golint77%

Golint is a linter for Go source code.

    • grgd/interfaces/IProject.go
    • Line 28: warning: exported type ProjectMetadata should have comment or be unexported (golint)
    • Line 38: warning: exported type ServiceLocator should have comment or be unexported (golint)
    • grgd/pkg/helper/Extractor.go
    • Line 46: warning: comment on exported method Extractor.GetMetadataFatal should be of the form "GetMetadataFatal ..." (golint)
    • Line 54: warning: comment on exported method Extractor.GetCore should be of the form "GetCore ..." (golint)
    • grgd/grgd.go
    • Line 16: warning: exported function NewApp should have comment or be unexported (golint)
    • grgd/plugins/grgdk3d/grgdk3d.go
    • Line 25: warning: exported type CMD should have comment or be unexported (golint)
    • Line 28: warning: exported method CMD.GetCommands should have comment or be unexported (golint)
    • Line 308: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • grgd/pkg/profile/profile.go
    • Line 172: warning: exported method Profile.AddProject should have comment or be unexported (golint)
    • Line 176: warning: exported method Profile.AddProjectByName should have comment or be unexported (golint)
    • Line 188: warning: comment on exported method Profile.AddProjectDirect should be of the form "AddProjectDirect ..." (golint)
    • Line 209: warning: comment on exported method Profile.GetCurrentProjectID should be of the form "GetCurrentProjectID ..." (golint)
    • Line 214: warning: comment on exported method Profile.SetCurrentProjectID should be of the form "SetCurrentProjectID ..." (golint)
    • Line 246: warning: exported method Profile.GetValuesAsMap should have comment or be unexported (golint)
    • grgd/pkg/helper/Helper.go
    • Line 130: warning: exported method Helper.ReadFile should have comment or be unexported (golint)
    • Line 134: warning: exported method Helper.UpdateOrWriteFile should have comment or be unexported (golint)
    • Line 157: warning: exported method Helper.CatchOutput should have comment or be unexported (golint)
    • grgd/core/core.go
    • Line 216: warning: comment on exported method Core.GetNetworker should be of the form "GetNetworker ..." (golint)
    • Line 236: warning: comment on exported method Core.CallPreHook should be of the form "CallPreHook ..." (golint)
    • Line 258: warning: comment on exported method Core.CallPostHook should be of the form "CallPostHook ..." (golint)
    • grgd/pkg/project/project.go
    • Line 173: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 178: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 184: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 191: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 196: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 202: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 207: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 212: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 218: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 249: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 255: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 260: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 266: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 270: warning: comment on exported method Project.WriteSettingsObject should be of the form "WriteSettingsObject ..." (golint)
    • Line 271: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • Line 280: warning: comment on exported method Project.ReadSettingsObject should be of the form "ReadSettingsObject ..." (golint)
    • Line 281: warning: receiver name project should be consistent with previous receiver name p for Project (golint)
    • grgd/pkg/helper/Updater.go
    • Line 16: warning: comment on exported function ProvideUpdater should be of the form "ProvideUpdater ..." (golint)
    • Line 25: warning: exported type Updater should have comment or be unexported (golint)
    • Line 167: warning: exported method Updater.CheckSinceLastUpdate should have comment or be unexported (golint)
    • grgd/interfaces/IService.go
    • Line 14: warning: exported type ServiceMetadata should have comment or be unexported (golint)
    • Line 25: warning: exported type RunTime should have comment or be unexported (golint)
    • Line 32: warning: exported type VersionHandle should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign92%

IneffAssign detects ineffectual assignments in Go code.

    • grgd/cmd/config/config_test.go
    • Line 6: warning: no required module provides package github.com/gregod-com/grgd/interfaces/mocks; to add it: (ineffassign)
    • Line 6: warning: could not import github.com/gregod-com/grgd/interfaces/mocks (invalid package name: "") (ineffassign)
    • grgd/pkg/config/ConfigDatabase_test.go
    • Line 6: warning: no required module provides package github.com/gregod-com/grgd/interfaces/mocks; to add it: (ineffassign)
    • Line 6: warning: could not import github.com/gregod-com/grgd/interfaces/mocks (invalid package name: "") (ineffassign)
    • grgd/pkg/gormdal/profileModel_test.go
    • Line 8: warning: no required module provides package github.com/gregod-com/grgd/interfaces/mocks; to add it: (ineffassign)
    • Line 8: warning: could not import github.com/gregod-com/grgd/interfaces/mocks (invalid package name: "") (ineffassign)
    • grgd/cmd/profile/profile_test.go
    • Line 6: warning: no required module provides package github.com/gregod-com/grgd/interfaces/mocks; to add it: (ineffassign)
    • Line 6: warning: could not import github.com/gregod-com/grgd/interfaces/mocks (invalid package name: "") (ineffassign)

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!