Preparing report...

Report for github.com/negrel/paon

A+    Excellent!    Found 16 issues across 61 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!


gocyclo100%

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.

No problems detected. Good job!


golint78%

Golint is a linter for Go source code.

    • paon/pdk/tree/node_test.go
    • Line 74: warning: don't use underscores in Go names; func Node_AppendChild_ToEmptyNode should be NodeAppendChildToEmptyNode (golint)
    • Line 88: warning: don't use underscores in Go names; func Node_AppendChild_ToNonEmptyNode should be NodeAppendChildToNonEmptyNode (golint)
    • Line 107: warning: don't use underscores in Go names; func Node_AppendChild_NilNode should be NodeAppendChildNilNode (golint)
    • Line 117: warning: don't use underscores in Go names; func Node_AppendChild_ParentOfNode should be NodeAppendChildParentOfNode (golint)
    • Line 129: warning: don't use underscores in Go names; func Node_AppendChild_GreatParentOfNode should be NodeAppendChildGreatParentOfNode (golint)
    • Line 146: warning: don't use underscores in Go names; func Node_AppendChild_NodeWithParent should be NodeAppendChildNodeWithParent (golint)
    • Line 162: warning: don't use underscores in Go names; func Node_AppendChild_Itself should be NodeAppendChildItself (golint)
    • Line 170: warning: don't use underscores in Go names; func Node_InsertBeforeNode should be NodeInsertBeforeNode (golint)
    • Line 191: warning: don't use underscores in Go names; func Node_InsertBeforeNode_NonChildReference should be NodeInsertBeforeNodeNonChildReference (golint)
    • Line 205: warning: don't use underscores in Go names; func Node_InsertBeforeNode_ToEmptyParent_NilReference should be NodeInsertBeforeNodeToEmptyParentNilReference (golint)
    • Line 219: warning: don't use underscores in Go names; func Node_InsertBeforeNode_ToEmptyParent_NilChild should be NodeInsertBeforeNodeToEmptyParentNilChild (golint)
    • Line 231: warning: don't use underscores in Go names; func Node_InsertBeforeNode_ParentOfNode should be NodeInsertBeforeNodeParentOfNode (golint)
    • Line 247: warning: don't use underscores in Go names; func Node_InsertBeforeNode_GreatParentOfNode should be NodeInsertBeforeNodeGreatParentOfNode (golint)
    • Line 266: warning: don't use underscores in Go names; func Node_InsertBeforeNode_ChildWithParent should be NodeInsertBeforeNodeChildWithParent (golint)
    • Line 290: warning: don't use underscores in Go names; func Node_InsertBeforeNode_BetweenTwoNode should be NodeInsertBeforeNodeBetweenTwoNode (golint)
    • Line 318: warning: don't use underscores in Go names; func Node_InsertBeforeNode_Itself should be NodeInsertBeforeNodeItself (golint)
    • Line 330: warning: don't use underscores in Go names; func Node_RemoveChild should be NodeRemoveChild (golint)
    • Line 354: warning: don't use underscores in Go names; func Node_RemoveChild_Nil should be NodeRemoveChildNil (golint)
    • Line 364: warning: don't use underscores in Go names; func Node_RemoveChild_AnotherParentChild should be NodeRemoveChildAnotherParentChild (golint)
    • Line 377: warning: don't use underscores in Go names; func Node_RemoveChild_SecondChild should be NodeRemoveChildSecondChild (golint)
    • paon/styles/property_change.go
    • Line 24: warning: exported function EventTypePropertyChange should have comment or be unexported (golint)
    • Line 28: warning: exported type EventPropertyChange should have comment or be unexported (golint)
    • paon/styles/property/int.go
    • Line 3: warning: exported type Int should have comment or be unexported (golint)
    • Line 8: warning: exported function NewInt should have comment or be unexported (golint)
    • paon/styles/property/property.go
    • Line 3: warning: exported type Property should have comment or be unexported (golint)
    • Line 7: warning: exported type Prop should have comment or be unexported (golint)
    • Line 11: warning: exported function NewProp should have comment or be unexported (golint)
    • Line 17: warning: exported method Prop.ID should have comment or be unexported (golint)
    • paon/pdk/tree/leaf_node_test.go
    • Line 100: warning: don't use underscores in Go names; func Node_Unwrap should be NodeUnwrap (golint)
    • Line 104: warning: don't use underscores in Go names; func LeafNode_FirstChild_Nil should be LeafNodeFirstChildNil (golint)
    • Line 108: warning: don't use underscores in Go names; func LeafNode_LastChild_Nil should be LeafNodeLastChildNil (golint)
    • Line 112: warning: don't use underscores in Go names; func Node_IsSame_true should be NodeIsSameTrue (golint)
    • Line 116: warning: don't use underscores in Go names; func Node_IsSame_false should be NodeIsSameFalse (golint)
    • Line 120: warning: don't use underscores in Go names; func Node_IsDescendantOf_NilParent should be NodeIsDescendantOfNilParent (golint)
    • Line 125: warning: don't use underscores in Go names; func Node_IsDescendantOf_Parent should be NodeIsDescendantOfParent (golint)
    • Line 132: warning: don't use underscores in Go names; func Node_IsDescendantOf_NonChildNode should be NodeIsDescendantOfNonChildNode (golint)
    • Line 139: warning: don't use underscores in Go names; func Node_IsDescendantOf_PreviousParent should be NodeIsDescendantOfPreviousParent (golint)
    • Line 151: warning: don't use underscores in Go names; func Node_IsDescendantOf_GreatParent should be NodeIsDescendantOfGreatParent (golint)
    • Line 164: warning: don't use underscores in Go names; func Node_Root_Nil should be NodeRootNil (golint)
    • Line 169: warning: don't use underscores in Go names; func Node_Root_Parent should be NodeRootParent (golint)
    • Line 176: warning: don't use underscores in Go names; func Node_Root_GreatParent should be NodeRootGreatParent (golint)
    • Line 189: warning: don't use underscores in Go names; func LeafNode_IsAncestorOf_Nil should be LeafNodeIsAncestorOfNil (golint)
    • Line 193: warning: don't use underscores in Go names; func LeafNode_AppendChild_LeafNodeError should be LeafNodeAppendChildLeafNodeError (golint)
    • Line 199: warning: don't use underscores in Go names; func LeafNode_InsertBefore_LeafNodeError should be LeafNodeInsertBeforeLeafNodeError (golint)
    • Line 208: warning: don't use underscores in Go names; func LeafNode_RemoveChild_LeafNodeError should be LeafNodeRemoveChildLeafNodeError (golint)
    • paon/styles/value/unit.go
    • Line 3: warning: exported type UnitID should have comment or be unexported (golint)
    • Line 6: warning: exported const CellUnit should have comment (or a comment on this block) or be unexported (golint)
    • Line 13: warning: exported type Unit should have comment or be unexported (golint)
    • paon/pdk/layout/box.go
    • Line 19: warning: exported type Box should have comment or be unexported (golint)
    • Line 78: warning: exported method Box.ApplyMargin should have comment or be unexported (golint)
    • Line 100: warning: exported method Box.ApplyBorder should have comment or be unexported (golint)
    • Line 108: warning: exported method Box.ApplyPadding should have comment or be unexported (golint)
    • paon/styles/options.go
    • Line 5: warning: exported type Option should have comment or be unexported (golint)
    • Line 7: warning: comment on exported function Weight should be of the form "Weight ..." (golint)
    • paon/styles/property/id.go
    • Line 22: warning: exported function NewID should have comment or be unexported (golint)
    • Line 60: warning: exported function Flow should have comment or be unexported (golint)
    • Line 64: warning: exported function Width should have comment or be unexported (golint)
    • Line 68: warning: exported function MinWidth should have comment or be unexported (golint)
    • Line 72: warning: exported function MaxWidth should have comment or be unexported (golint)
    • Line 76: warning: exported function Height should have comment or be unexported (golint)
    • Line 80: warning: exported function MinHeight should have comment or be unexported (golint)
    • Line 84: warning: exported function MaxHeight should have comment or be unexported (golint)
    • Line 88: warning: exported function MarginLeft should have comment or be unexported (golint)
    • Line 92: warning: exported function MarginTop should have comment or be unexported (golint)
    • Line 96: warning: exported function MarginRight should have comment or be unexported (golint)
    • Line 100: warning: exported function MarginBottom should have comment or be unexported (golint)
    • Line 104: warning: exported function BorderLeft should have comment or be unexported (golint)
    • Line 108: warning: exported function BorderTop should have comment or be unexported (golint)
    • Line 112: warning: exported function BorderRight should have comment or be unexported (golint)
    • Line 116: warning: exported function BorderBottom should have comment or be unexported (golint)
    • Line 120: warning: exported function BorderCharset should have comment or be unexported (golint)
    • Line 124: warning: exported function BorderColor should have comment or be unexported (golint)
    • Line 128: warning: exported function PaddingLeft should have comment or be unexported (golint)
    • Line 132: warning: exported function PaddingTop should have comment or be unexported (golint)
    • Line 136: warning: exported function PaddingRight should have comment or be unexported (golint)
    • Line 140: warning: exported function PaddingBottom should have comment or be unexported (golint)
    • Line 144: warning: exported function BackgroundColor should have comment or be unexported (golint)
    • Line 148: warning: exported function ForegroundColor should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign96%

IneffAssign detects ineffectual assignments in Go code.

    • paon/pdk/draw/context_test.go
    • Line 16: warning: undeclared name: NewMockCanvas (ineffassign)
    • Line 32: warning: undeclared name: NewMockCanvas (ineffassign)
    • Line 57: warning: undeclared name: NewMockCanvas (ineffassign)
    • Line 87: warning: undeclared name: NewMockCanvas (ineffassign)
    • Line 117: warning: undeclared name: NewMockCanvas (ineffassign)
    • Line 149: warning: undeclared name: NewMockCanvas (ineffassign)

misspell98%

Misspell Finds commonly misspelled English words