gofmt80%

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

• config-lint/cli/app_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/linter/file_linter_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/linter/terraform_v12.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/assertion/expression_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/assertion/rules.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/assertion/util_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/linter/tf12parser/parser.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/linter/tf12parser/parser_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/cli/report_writer_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/assertion/invoke_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/assertion/rules_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• config-lint/linter/tf12parser/block.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!

golint80%

Golint is a linter for Go source code.

• config-lint/linter/tf12parser/parser.go
  • Line 31: warning: exported method Parser.ParseMany should have comment or be unexported (golint)
  • Line 76: warning: exported method Parser.ParseFile should have comment or be unexported (golint)

• config-lint/cli/report_writer.go
  • Line 8: warning: exported method DefaultReportWriter.WriteReport should have comment or be unexported (golint)

• config-lint/linter/linter.go
  • Line 34: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 40: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• config-lint/linter/terraform_v12_test.go
  • Line 438: warning: don't use underscores in Go names; var properties_1 should be properties1 (golint)
  • Line 440: warning: don't use underscores in Go names; var properties_2 should be properties2 (golint)

• config-lint/assertion/util.go
  • Line 151: warning: exported function SliceContains should have comment or be unexported (golint)
  • Line 160: warning: comment on exported function ExcludeResourceTypes should be of the form "ExcludeResourceTypes ..." (golint)

• config-lint/linter/tf12parser/attribute.go
  • Line 9: warning: exported type Attribute should have comment or be unexported (golint)
  • Line 14: warning: exported function NewAttribute should have comment or be unexported (golint)
  • Line 21: warning: exported method Attribute.IsLiteral should have comment or be unexported (golint)
  • Line 25: warning: exported method Attribute.Type should have comment or be unexported (golint)
  • Line 29: warning: exported method Attribute.Value should have comment or be unexported (golint)
  • Line 40: warning: exported method Attribute.Range should have comment or be unexported (golint)
  • Line 48: warning: exported method Attribute.Name should have comment or be unexported (golint)

• config-lint/linter/tf12parser/block.go
  • Line 10: warning: exported type Block should have comment or be unexported (golint)
  • Line 16: warning: exported type Blocks should have comment or be unexported (golint)
  • Line 18: warning: exported method Blocks.OfType should have comment or be unexported (golint)
  • Line 28: warning: exported method Blocks.RemoveDuplicates should have comment or be unexported (golint)
  • Line 45: warning: exported function NewBlock should have comment or be unexported (golint)
  • Line 56: warning: exported method Block.Type should have comment or be unexported (golint)
  • Line 60: warning: exported method Block.Labels should have comment or be unexported (golint)
  • Line 64: warning: exported method Block.Range should have comment or be unexported (golint)
  • Line 76: warning: exported method Block.AllBlocks should have comment or be unexported (golint)
  • Line 87: warning: exported method Block.GetBlock should have comment or be unexported (golint)
  • Line 99: warning: exported method Block.GetBlocks should have comment or be unexported (golint)
  • Line 112: warning: exported method Block.GetAttributes should have comment or be unexported (golint)
  • Line 123: warning: exported method Block.GetAttribute should have comment or be unexported (golint)
  • Line 135: warning: exported method Block.Name should have comment or be unexported (golint)

• config-lint/cli/terraform_test.go
  • Line 39: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• config-lint/linter/terraform_interpolate.go
  • Line 109: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 375: warning: exported function Funcs should have comment or be unexported (golint)

• config-lint/linter/terraform_v12.go
  • Line 64: warning: exported method Terraform12ResourceLoader.LoadMany should have comment or be unexported (golint)
  • Line 241: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 248: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• config-lint/assertion/log.go
  • Line 22: warning: exported function DebugJSON should have comment or be unexported (golint)

• config-lint/assertion/rules.go
  • Line 169: warning: comment on exported function JoinRuleSets should be of the form "JoinRuleSets ..." (golint)
  • Line 183: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

gocyclo96%

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.

• config-lint/linter/tf12parser/parser.go
  • Line 278: warning: cyclomatic complexity 19 of function (*Parser).getValuesByBlockType() is high (> 15) (gocyclo)

• config-lint/assertion/match.go
  • Line 27: warning: cyclomatic complexity 57 of function isMatch() is high (> 15) (gocyclo)

ineffassign95%

IneffAssign detects ineffectual assignments in Go code.

• config-lint/cli/options_test.go
  • Line 163: warning: ineffectual assignment to parser (ineffassign)

• config-lint/cli/terraform_test.go
  • Line 117: warning: ineffectual assignment to err (ineffassign)
  • Line 140: warning: ineffectual assignment to err (ineffassign)

• config-lint/assertion/match_test.go
  • Line 140: warning: ineffectual assignment to err (ineffassign)

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!