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!

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.

• faas-netes/pkg/k8s/profiles.go
  • Line 181: warning: cyclomatic complexity 17 of function (FunctionFactory).RemoveProfile() is high (> 15) (gocyclo)

• faas-netes/pkg/handlers/secrets_test.go
  • Line 25: warning: cyclomatic complexity 20 of function Test_SecretsHandler() is high (> 15) (gocyclo)

• faas-netes/pkg/handlers/update.go
  • Line 76: warning: cyclomatic complexity 18 of function updateDeploymentSpec() is high (> 15) (gocyclo)

• faas-netes/pkg/controller/controller.go
  • Line 215: warning: cyclomatic complexity 19 of function (*Controller).syncHandler() is high (> 15) (gocyclo)
  • Line 407: warning: cyclomatic complexity 18 of function getReplicas() is high (> 15) (gocyclo)

golint84%

Golint is a linter for Go source code.

• faas-netes/pkg/handlers/update.go
  • Line 76: warning: error should be the last type when returning multiple items (golint)
  • Line 202: warning: error should be the last type when returning multiple items (golint)

• faas-netes/pkg/apis/openfaas/register.go
  • Line 4: warning: exported const GroupName should have comment (or a comment on this block) or be unexported (golint)

• faas-netes/pkg/apis/openfaas/v1/register.go
  • Line 20: warning: comment on exported var SchemeBuilder should be of the form "SchemeBuilder ..." (golint)
  • Line 23: warning: exported var AddToScheme should have comment or be unexported (golint)

• faas-netes/pkg/config/read_config.go
  • Line 1: warning: package comment should be of the form "Package config ..." (golint)

• faas-netes/pkg/controller/controller.go
  • Line 37: warning: exported const LabelMinReplicas should have comment (or a comment on this block) or be unexported (golint)

• faas-netes/pkg/controller/deployment.go
  • Line 275: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• faas-netes/pkg/controller/factory.go
  • Line 19: warning: exported function NewFunctionFactory should have comment or be unexported (golint)
  • Line 64: warning: exported method FunctionFactory.MakeProbes should have comment or be unexported (golint)
  • Line 69: warning: exported method FunctionFactory.ConfigureReadOnlyRootFilesystem should have comment or be unexported (golint)
  • Line 74: warning: exported method FunctionFactory.ConfigureContainerUserID should have comment or be unexported (golint)
  • Line 78: warning: exported method FunctionFactory.ApplyProfile should have comment or be unexported (golint)
  • Line 82: warning: exported method FunctionFactory.RemoveProfile should have comment or be unexported (golint)
  • Line 86: warning: exported method FunctionFactory.GetProfiles should have comment or be unexported (golint)
  • Line 90: warning: exported method FunctionFactory.GetProfilesToRemove should have comment or be unexported (golint)

• faas-netes/pkg/server/server.go
  • Line 5: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
  • Line 90: warning: exported type Server should have comment or be unexported (golint)

• faas-netes/pkg/k8s/profiles.go
  • Line 17: warning: exported const ProfileAnnotationKey should have comment or be unexported (golint)
  • Line 81: warning: comment on exported method FunctionFactory.NewConfigMapProfileClient should be of the form "NewConfigMapProfileClient ..." (golint)
  • Line 100: warning: exported method FunctionFactory.GetProfilesToRemove should have comment or be unexported (golint)

• faas-netes/pkg/k8s/proxy.go
  • Line 20: warning: exported function NewFunctionLookup should have comment or be unexported (golint)
  • Line 29: warning: exported type FunctionLookup should have comment or be unexported (golint)
  • Line 37: warning: exported method FunctionLookup.GetLister should have comment or be unexported (golint)
  • Line 43: warning: exported method FunctionLookup.SetLister should have comment or be unexported (golint)
  • Line 57: warning: exported method FunctionLookup.Resolve should have comment or be unexported (golint)
  • Line 57: warning: receiver name l should be consistent with previous receiver name f for FunctionLookup (golint)
  • Line 104: warning: receiver name l should be consistent with previous receiver name f for FunctionLookup (golint)

• faas-netes/pkg/k8s/securityContext.go
  • Line 12: warning: comment on exported const SecurityContextUserID should be of the form "SecurityContextUserID ..." (golint)

• faas-netes/pkg/k8s/errors.go
  • Line 10: warning: comment on exported function IsNotFound should be of the form "IsNotFound ..." (golint)

• faas-netes/pkg/k8s/factory.go
  • Line 24: warning: exported function NewFunctionFactory should have comment or be unexported (golint)

• faas-netes/pkg/k8s/probes.go
  • Line 17: warning: exported const ProbePath should have comment (or a comment on this block) or be unexported (golint)
  • Line 22: warning: exported type FunctionProbes 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.

• faas-netes/pkg/controller/controller.go
  • Line 272: warning: ineffectual assignment to err (ineffassign)

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!