gofmt98%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command

• external/html/atom/gen.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

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!

gocyclo86%

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.

• external/html/escape.go
  • Line 57: warning: cyclomatic complexity 43 of function unescapeEntity() is high (> 15) (gocyclo)

• external/html/atom/gen.go
  • Line 57: warning: cyclomatic complexity 37 of function main() is high (> 15) (gocyclo)

• external/html/node_test.go
  • Line 34: warning: cyclomatic complexity 35 of function checkNodeConsistency() is high (> 15) (gocyclo)

• external/html/parse_test.go
  • Line 30: warning: cyclomatic complexity 33 of function readParseTest() is high (> 15) (gocyclo)
  • Line 153: warning: cyclomatic complexity 21 of function dumpLevel() is high (> 15) (gocyclo)
  • Line 291: warning: cyclomatic complexity 16 of function testParseCase() is high (> 15) (gocyclo)

• external/html/doctype.go
  • Line 16: warning: cyclomatic complexity 24 of function parseDoctype() is high (> 15) (gocyclo)

• external/html/charset/charset.go
  • Line 139: warning: cyclomatic complexity 22 of function prescan() is high (> 15) (gocyclo)
  • Line 52: warning: cyclomatic complexity 17 of function DetermineEncoding() is high (> 15) (gocyclo)

• reviewers/page_tag_parity.go
  • Line 49: warning: cyclomatic complexity 17 of function (PageTagParity).Review() is high (> 15) (gocyclo)

• external/html/token.go
  • Line 404: warning: cyclomatic complexity 50 of function (*Tokenizer).readScript() is high (> 15) (gocyclo)
  • Line 1052: warning: cyclomatic complexity 33 of function (*Tokenizer).Next() is high (> 15) (gocyclo)
  • Line 961: warning: cyclomatic complexity 21 of function (*Tokenizer).readTagAttrVal() is high (> 15) (gocyclo)

• reviewers/page_title_present.go
  • Line 28: warning: cyclomatic complexity 16 of function (PageTitlePresent).Review() is high (> 15) (gocyclo)

• external/html/render.go
  • Line 68: warning: cyclomatic complexity 57 of function render1() is high (> 15) (gocyclo)

• external/html/parse.go
  • Line 857: warning: cyclomatic complexity 109 of function inBodyIM() is high (> 15) (gocyclo)
  • Line 1411: warning: cyclomatic complexity 30 of function inTableIM() is high (> 15) (gocyclo)
  • Line 1205: warning: cyclomatic complexity 26 of function (*parser).inBodyEndTagFormatting() is high (> 15) (gocyclo)
  • Line 1780: warning: cyclomatic complexity 26 of function inSelectIM() is high (> 15) (gocyclo)
  • Line 636: warning: cyclomatic complexity 23 of function inHeadIM() is high (> 15) (gocyclo)
  • Line 2157: warning: cyclomatic complexity 23 of function parseForeignContent() is high (> 15) (gocyclo)
  • Line 447: warning: cyclomatic complexity 23 of function (*parser).resetInsertionMode() is high (> 15) (gocyclo)
  • Line 110: warning: cyclomatic complexity 21 of function (*parser).indexOfElementInScope() is high (> 15) (gocyclo)
  • Line 1564: warning: cyclomatic complexity 17 of function inColumnGroupIM() is high (> 15) (gocyclo)
  • Line 162: warning: cyclomatic complexity 16 of function (*parser).clearStackToContext() is high (> 15) (gocyclo)
  • Line 2250: warning: cyclomatic complexity 16 of function (*parser).inForeignContent() is high (> 15) (gocyclo)

golint77%

Golint is a linter for Go source code.

• external/html/render.go
  • Line 58: warning: error var plaintextAbort should have name of the form errFoo (golint)
  • Line 255: warning: comment on exported var VoidElements should be of the form "VoidElements ..." (golint)

• external/html/token.go
  • Line 25: warning: comment on exported const StartTagToken should be of the form "StartTagToken ..." (golint)
  • Line 27: warning: comment on exported const EndTagToken should be of the form "EndTagToken ..." (golint)
  • Line 29: warning: comment on exported const SelfClosingTagToken should be of the form "SelfClosingTagToken ..." (golint)
  • Line 31: warning: comment on exported const CommentToken should be of the form "CommentToken ..." (golint)
  • Line 33: warning: comment on exported const DoctypeToken should be of the form "DoctypeToken ..." (golint)

• cmd/command.go
  • Line 1: warning: package comment should be of the form "Package cmd ..." (golint)

• cmd/milo/review/review.go
  • Line 21: warning: exported var ErrFaultsFound should have comment or be unexported (golint)
  • Line 34: warning: exported method Runner.Name should have comment or be unexported (golint)
  • Line 38: warning: exported method Runner.HelpText should have comment or be unexported (golint)
  • Line 42: warning: exported method Runner.Run should have comment or be unexported (golint)

• reviewers/attribute_style_forbidden.go
  • Line 9: warning: exported type AttributeStyleForbidden should have comment or be unexported (golint)
  • Line 11: warning: exported method AttributeStyleForbidden.ReviewerName should have comment or be unexported (golint)
  • Line 15: warning: exported method AttributeStyleForbidden.Accepts should have comment or be unexported (golint)
  • Line 19: warning: exported method AttributeStyleForbidden.Review should have comment or be unexported (golint)

• external/html/node.go
  • Line 15: warning: exported const ErrorNode should have comment (or a comment on this block) or be unexported (golint)

• output/silent.go
  • Line 8: warning: exported method SilentFaultFormatter.FormatterName should have comment or be unexported (golint)
  • Line 12: warning: exported method SilentFaultFormatter.Format should have comment or be unexported (golint)

• output/text.go
  • Line 14: warning: exported method TextFaultFormatter.FormatterName should have comment or be unexported (golint)
  • Line 18: warning: exported method TextFaultFormatter.Format should have comment or be unexported (golint)

• reviewers/attribute_value_double_quotes.go
  • Line 13: warning: comment on exported method AttributeValueDoubleQuotes.Accepts should be of the form "Accepts ..." (golint)
  • Line 18: warning: comment on exported method AttributeValueDoubleQuotes.ReviewerName should be of the form "ReviewerName ..." (golint)

• output/github.go
  • Line 13: warning: exported method GithubFaultFormatter.FormatterName should have comment or be unexported (golint)
  • Line 17: warning: exported method GithubFaultFormatter.Format should have comment or be unexported (golint)

• output/output.go
  • Line 1: warning: package comment should be of the form "Package output ..." (golint)
  • Line 14: warning: comment on exported type FaultFormatter should be of the form "FaultFormatter ..." (with optional leading article) (golint)
  • Line 20: warning: exported function Formatter should have comment or be unexported (golint)

• reviewers/page_doctype_valid.go
  • Line 17: warning: comment on exported type PageDoctypeValid should be of the form "PageDoctypeValid ..." (with optional leading article) (golint)

• reviewers/rule.go
  • Line 3: warning: exported type Rule should have comment or be unexported (golint)

• reviewers/page_tag_lowercase_required.go
  • Line 9: warning: comment on exported type PageTagLowercaseRequired should be of the form "PageTagLowercaseRequired ..." (with optional leading article) (golint)

• cmd/milo/initialize/initializer.go
  • Line 1: warning: package comment should be of the form "Package initialize ..." (golint)
  • Line 13: warning: exported type Runner should have comment or be unexported (golint)
  • Line 15: warning: exported method Runner.Name should have comment or be unexported (golint)
  • Line 19: warning: exported method Runner.HelpText should have comment or be unexported (golint)
  • Line 23: warning: exported method Runner.Run should have comment or be unexported (golint)

• config/config.go
  • Line 1: warning: package comment should be of the form "Package config ..." (golint)
  • Line 15: warning: exported var ErrConfigNotFound should have comment or be unexported (golint)
  • Line 25: warning: exported method Settings.SelectedReviewers should have comment or be unexported (golint)
  • Line 42: warning: exported function Load should have comment or be unexported (golint)

• reviewers/attribute_lowercase.go
  • Line 22: warning: comment on exported method AttributeLowercase.Review should be of the form "Review ..." (golint)

• reviewers/fault.go
  • Line 3: warning: exported type Fault should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign98%

IneffAssign detects ineffectual assignments in Go code.

• config/config_test.go
  • Line 37: warning: ineffectual assignment to err (ineffassign)
  • Line 44: warning: ineffectual assignment to err (ineffassign)

misspell98%

Misspell Finds commonly misspelled English words

• external/html/entity.go
  • Line 614: warning: "andd" is a misspelling of "and" (misspell)
  • Line 1129: warning: "infintie" is a misspelling of "infinite" (misspell)