Preparing report...

Report for github.com/tc-hib/winres

A+    Excellent!    Found 11 issues across 28 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!


golint75%

Golint is a linter for Go source code.

    • winres/version/json.go
    • Line 23: warning: exported method Info.MarshalJSON should have comment or be unexported (golint)
    • Line 73: warning: exported method Info.UnmarshalJSON should have comment or be unexported (golint)
    • winres/version/version.go
    • Line 15: warning: exported const Comments should have comment (or a comment on this block) or be unexported (golint)
    • Line 32: warning: exported const App should have comment (or a comment on this block) or be unexported (golint)
    • winres/exe.go
    • Line 15: warning: exported const ErrorIfSigned should have comment (or a comment on this block) or be unexported (golint)
    • winres/icon.go
    • Line 25: warning: exported var DefaultIconSizes should have comment or be unexported (golint)
    • winres/manifest.go
    • Line 54: warning: exported const DPIAware should have comment (or a comment on this block) or be unexported (golint)
    • Line 69: warning: exported const WinVistaAndAbove should have comment (or a comment on this block) or be unexported (golint)
    • Line 80: warning: exported const AsInvoker should have comment (or a comment on this block) or be unexported (golint)
    • Line 398: warning: exported method SupportedOS.MarshalText should have comment or be unexported (golint)
    • Line 414: warning: exported method SupportedOS.UnmarshalText should have comment or be unexported (golint)
    • Line 435: warning: exported method DPIAwareness.MarshalText should have comment or be unexported (golint)
    • Line 449: warning: exported method DPIAwareness.UnmarshalText should have comment or be unexported (golint)
    • Line 467: warning: exported method ExecutionLevel.MarshalText should have comment or be unexported (golint)
    • Line 479: warning: exported method ExecutionLevel.UnmarshalText should have comment or be unexported (golint)
    • Line 499: warning: exported method AssemblyIdentity.MarshalJSON should have comment or be unexported (golint)
    • Line 511: warning: exported method AssemblyIdentity.UnmarshalJSON should have comment or be unexported (golint)
    • winres/winres.go
    • Line 37: warning: exported const LCIDNeutral should have comment (or a comment on this block) or be unexported (golint)
    • Line 47: warning: exported const ArchI386 should have comment (or a comment on this block) or be unexported (golint)

gocyclo78%

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.

    • winres/winres_test.go
    • Line 682: warning: cyclomatic complexity 43 of function TestResourceSet_LoadFromEXE_Err() is high (> 15) (gocyclo)
    • Line 18: warning: cyclomatic complexity 18 of function TestErrors() is high (> 15) (gocyclo)
    • winres/icon_test.go
    • Line 477: warning: cyclomatic complexity 26 of function TestResourceSet_GetIconTranslation_Err() is high (> 15) (gocyclo)
    • winres/cursor_test.go
    • Line 422: warning: cyclomatic complexity 26 of function TestResourceSet_GetCursorTranslation_Err() is high (> 15) (gocyclo)
    • winres/exe.go
    • Line 470: warning: cyclomatic complexity 16 of function (*peWriter).writeEXE() is high (> 15) (gocyclo)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!