gofmt97%

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

• carbon/operator/builtin/input/azure/event_hub_parse.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• carbon/operator/builtin/input/azure/event_hub_parse_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• carbon/operator/builtin/transformer/k8smetadata/k8s_metadata_decorator.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• carbon/operator/builtin/parser/uri/uri.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• carbon/operator/builtin/parser/uri/uri_test.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!

gocyclo98%

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.

• carbon/operator/builtin/input/file/config.go
  • Line 67: warning: cyclomatic complexity 18 of function (InputConfig).Build() is high (> 15) (gocyclo)

• carbon/operator/builtin/output/otlp/converter.go
  • Line 148: warning: cyclomatic complexity 24 of function toAttributeMap() is high (> 15) (gocyclo)
  • Line 92: warning: cyclomatic complexity 23 of function insertToAttributeVal() is high (> 15) (gocyclo)

• carbon/operator/builtin/output/googlecloud/google_cloud.go
  • Line 119: warning: cyclomatic complexity 16 of function (*GoogleCloudOutput).Start() is high (> 15) (gocyclo)

• carbon/entry/field.go
  • Line 102: warning: cyclomatic complexity 22 of function splitField() is high (> 15) (gocyclo)

golint91%

Golint is a linter for Go source code.

• carbon/plugin/config.go
  • Line 25: warning: comment on exported method Config.Build should be of the form "Build ..." (golint)
  • Line 74: warning: exported method Config.UnmarshalJSON should have comment or be unexported (golint)

• carbon/operator/builtin/input/journald/journald.go
  • Line 27: warning: exported function NewJournaldInputConfig should have comment or be unexported (golint)

• carbon/operator/builtin/input/azure/event_hub_config.go
  • Line 27: warning: exported method AzureConfig.Build should have comment or be unexported (golint)

• carbon/version/version.go
  • Line 6: warning: exported var GitCommit should have comment or be unexported (golint)

• carbon/operator/builtin/output/newrelic/newrelic.go
  • Line 239: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• carbon/operator/builtin/output/googlecloud/google_cloud.go
  • Line 238: warning: exported method GoogleCloudOutput.Send should have comment or be unexported (golint)
  • Line 244: warning: exported method GoogleCloudOutput.IsTooLargeError should have comment or be unexported (golint)

• carbon/operator/builtin/input/azure/event_hub.go
  • Line 12: warning: comment on exported type EventHub should be of the form "EventHub ..." (with optional leading article) (golint)

• carbon/operator/buffer/buffer.go
  • Line 74: warning: exported method Config.MarshalYAML should have comment or be unexported (golint)
  • Line 78: warning: exported method Config.MarshalJSON should have comment or be unexported (golint)
  • Line 82: warning: exported type Clearer should have comment or be unexported (golint)

• carbon/operator/builtin/output/forward/forward.go
  • Line 167: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• carbon/operator/helper/parser.go
  • Line 79: warning: exported method ParserOperator.ProcessWithCallback should have comment or be unexported (golint)

• carbon/operator/helper/writer.go
  • Line 107: warning: exported method OutputIDs.WithNamespace should have comment or be unexported (golint)

• carbon/operator/builtin/input/aws/cloudwatch/cloudwatch_persist.go
  • Line 10: warning: exported type Persister should have comment or be unexported (golint)

• carbon/operator/buffer/disk.go
  • Line 318: warning: exported method DiskBuffer.MaxChunkSize should have comment or be unexported (golint)
  • Line 324: warning: exported method DiskBuffer.MaxChunkDelay should have comment or be unexported (golint)
  • Line 330: warning: exported method DiskBuffer.SetMaxChunkSize should have comment or be unexported (golint)
  • Line 336: warning: exported method DiskBuffer.SetMaxChunkDelay should have comment or be unexported (golint)

• carbon/operator/helper/transformer.go
  • Line 104: warning: exported method TransformerOperator.Skip should have comment or be unexported (golint)

• carbon/operator/builtin/input/azure/event_hub_parse.go
  • Line 44: warning: should omit 2nd value from range; this loop is equivalent to `for key := range ...` (golint)

• carbon/operator/buffer/memory.go
  • Line 149: warning: exported method MemoryBuffer.MaxChunkSize should have comment or be unexported (golint)
  • Line 155: warning: exported method MemoryBuffer.MaxChunkDelay should have comment or be unexported (golint)
  • Line 161: warning: exported method MemoryBuffer.SetMaxChunkSize should have comment or be unexported (golint)
  • Line 167: warning: exported method MemoryBuffer.SetMaxChunkDelay should have comment or be unexported (golint)

• carbon/operator/builtin/transformer/recombine/recombine.go
  • Line 111: warning: exported method RecombineOperator.Start should have comment or be unexported (golint)
  • Line 115: warning: exported method RecombineOperator.Stop should have comment or be unexported (golint)
  • Line 125: warning: exported method RecombineOperator.Process should have comment or be unexported (golint)

• carbon/operator/builtin/output/otlp/otlp.go
  • Line 183: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• carbon/operator/helper/bytesize.go
  • Line 11: warning: exported type ByteSize should have comment or be unexported (golint)
  • Line 13: warning: exported method ByteSize.UnmarshalJSON should have comment or be unexported (golint)
  • Line 19: warning: exported method ByteSize.UnmarshalYAML 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.

• carbon/operator/builtin/output/otlp/converter.go
  • Line 77: warning: ineffectual assignment to resourceEntries (ineffassign)

• carbon/operator/builtin/input/azure/loganalytics/parse.go
  • Line 33: warning: ineffectual assignment to id (ineffassign)

• carbon/operator/helper/persister_test.go
  • Line 22: warning: ineffectual assignment to err (ineffassign)

misspell98%

Misspell Finds commonly misspelled English words

• carbon/operator/builtin/input/azure/eventhub/parse.go
  • Line 11: warning: "recieved" is a misspelling of "received" (misspell)

• carbon/operator/builtin/input/azure/loganalytics/parse.go
  • Line 18: warning: "recieved" is a misspelling of "received" (misspell)
  • Line 45: warning: "gaurd" is a misspelling of "guard" (misspell)
  • Line 49: warning: "gaurd" is a misspelling of "guard" (misspell)
  • Line 53: warning: "gaurd" is a misspelling of "guard" (misspell)

• carbon/operator/helper/time.go
  • Line 176: warning: "psuedo" is a misspelling of "pseudo" (misspell)