gofmt92%

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

• jetstack-secure/api/cluster_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• jetstack-secure/pkg/agent/config_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• jetstack-secure/pkg/datagatherer/k8s/dynamic_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!

gocyclo95%

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.

• jetstack-secure/pkg/datagatherer/k8s/dynamic_test.go
  • Line 279: warning: cyclomatic complexity 18 of function TestDynamicGatherer_Fetch() is high (> 15) (gocyclo)

• jetstack-secure/pkg/datagatherer/versionchecker/versionchecker.go
  • Line 67: warning: cyclomatic complexity 21 of function (*Config).UnmarshalYAML() is high (> 15) (gocyclo)
  • Line 315: warning: cyclomatic complexity 20 of function (*DataGatherer).Fetch() is high (> 15) (gocyclo)

golint73%

Golint is a linter for Go source code.

• jetstack-secure/pkg/datagatherer/versionchecker/versionchecker.go
  • Line 300: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)

• jetstack-secure/pkg/datagatherer/eks/eks.go
  • Line 52: warning: exported method DataGatherer.Run should have comment or be unexported (golint)
  • Line 57: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)

• jetstack-secure/pkg/datagatherer/gke/gke.go
  • Line 88: warning: exported method DataGatherer.Run should have comment or be unexported (golint)
  • Line 93: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)
  • Line 98: warning: exported method DataGatherer.WaitForCacheSync should have comment or be unexported (golint)

• jetstack-secure/api/datareading.go
  • Line 35: warning: exported method GatheredResource.MarshalJSON should have comment or be unexported (golint)

• jetstack-secure/pkg/datagatherer/aks/aks.go
  • Line 109: warning: exported method DataGatherer.Run should have comment or be unexported (golint)
  • Line 114: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)
  • Line 119: warning: exported method DataGatherer.WaitForCacheSync should have comment or be unexported (golint)

• jetstack-secure/pkg/agent/config.go
  • Line 40: warning: exported type Endpoint should have comment or be unexported (golint)

• jetstack-secure/pkg/agent/dummy_data_gatherer.go
  • Line 47: warning: receiver name c should be consistent with previous receiver name g for dummyDataGatherer (golint)

• jetstack-secure/pkg/datagatherer/local/local.go
  • Line 41: warning: exported method DataGatherer.Run should have comment or be unexported (golint)
  • Line 46: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)
  • Line 51: warning: exported method DataGatherer.WaitForCacheSync should have comment or be unexported (golint)

• jetstack-secure/pkg/client/client.go
  • Line 14: warning: comment on exported var ClientID should be of the form "ClientID ..." (golint)
  • Line 16: warning: exported var ClientSecret should have comment or be unexported (golint)
  • Line 17: warning: exported var AuthServerDomain should have comment or be unexported (golint)

• jetstack-secure/pkg/datagatherer/k8s/discovery.go
  • Line 49: warning: exported method DataGathererDiscovery.Run should have comment or be unexported (golint)
  • Line 54: warning: exported method DataGathererDiscovery.WaitForCacheSync should have comment or be unexported (golint)
  • Line 59: warning: exported method DataGathererDiscovery.Delete should have comment or be unexported (golint)

• jetstack-secure/pkg/datagatherer/istio/istio.go
  • Line 1: warning: package comment should be of the form "Package istio ..." (golint)
  • Line 130: warning: exported method DataGatherer.Delete should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign0%

IneffAssign detects ineffectual assignments in Go code.

An error occurred while running this test (signal: killed)

misspell92%

Misspell Finds commonly misspelled English words

• jetstack-secure/cmd/agent.go
  • Line 59: warning: "continously" is a misspelling of "continuously" (misspell)

• jetstack-secure/pkg/agent/run.go
  • Line 120: warning: "inital" is a misspelling of "initial" (misspell)
  • Line 125: warning: "inital" is a misspelling of "initial" (misspell)
  • Line 127: warning: "inital" is a misspelling of "initial" (misspell)

• jetstack-secure/api/datareading.go
  • Line 18: warning: "infered" is a misspelling of "inferred" (misspell)