gofmt97%

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

• terraboard/internal/terraform/configs/configschema/decoder_spec_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• terraboard/internal/terraform/configs/configschema/implied_type_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• terraboard/internal/terraform/configs/configschema/internal_validate_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• terraboard/compare/compare_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• terraboard/internal/terraform/configs/configload/loader_snapshot.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• terraboard/internal/terraform/configs/hcl2shim/paths_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!

gocyclo90%

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.

• terraboard/internal/terraform/configs/configschema/coerce_value.go
  • Line 29: warning: cyclomatic complexity 54 of function (*Block).coerceValue() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/module.go
  • Line 167: warning: cyclomatic complexity 24 of function (*Module).appendFile() is high (> 15) (gocyclo)
  • Line 334: warning: cyclomatic complexity 21 of function (*Module).mergeFile() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/module_call.go
  • Line 33: warning: cyclomatic complexity 18 of function decodeModuleBlock() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/named_values.go
  • Line 36: warning: cyclomatic complexity 16 of function decodeVariableBlock() is high (> 15) (gocyclo)

• terraboard/internal/terraform/tfdiags/contextual.go
  • Line 205: warning: cyclomatic complexity 21 of function traversePathSteps() is high (> 15) (gocyclo)

• terraboard/internal/terraform/typeexpr/get_type.go
  • Line 15: warning: cyclomatic complexity 35 of function getType() is high (> 15) (gocyclo)

• terraboard/internal/terraform/typeexpr/public.go
  • Line 44: warning: cyclomatic complexity 19 of function TypeString() is high (> 15) (gocyclo)

• terraboard/internal/terraform/addrs/provider.go
  • Line 233: warning: cyclomatic complexity 17 of function ParseProviderSourceString() is high (> 15) (gocyclo)

• terraboard/internal/terraform/addrs/module_instance.go
  • Line 394: warning: cyclomatic complexity 17 of function (ModuleInstance).TargetContains() is high (> 15) (gocyclo)
  • Line 80: warning: cyclomatic complexity 17 of function parseModuleInstancePrefix() is high (> 15) (gocyclo)

• terraboard/internal/terraform/registry/client.go
  • Line 185: warning: cyclomatic complexity 16 of function (*Client).ModuleLocation() is high (> 15) (gocyclo)

• terraboard/internal/terraform/states/statefile/version4.go
  • Line 32: warning: cyclomatic complexity 37 of function prepareStateV4() is high (> 15) (gocyclo)
  • Line 298: warning: cyclomatic complexity 17 of function writeStateV4() is high (> 15) (gocyclo)

• terraboard/internal/terraform/tfdiags/diagnostics.go
  • Line 47: warning: cyclomatic complexity 18 of function (Diagnostics).Append() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/provider_requirements.go
  • Line 30: warning: cyclomatic complexity 31 of function decodeRequiredProvidersBlock() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/parser_config.go
  • Line 32: warning: cyclomatic complexity 26 of function (*Parser).loadConfigFile() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/provisioner.go
  • Line 22: warning: cyclomatic complexity 18 of function decodeProvisionerBlock() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/hcl2shim/values.go
  • Line 35: warning: cyclomatic complexity 20 of function ConfigValueFromHCL2Block() is high (> 15) (gocyclo)
  • Line 127: warning: cyclomatic complexity 17 of function ConfigValueFromHCL2() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/config_build_test.go
  • Line 159: warning: cyclomatic complexity 27 of function TestBuildConfigInvalidModules() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/hcl2shim/flatmap.go
  • Line 340: warning: cyclomatic complexity 19 of function hcl2ValueFromFlatmapSet() is high (> 15) (gocyclo)

• terraboard/internal/terraform/states/statemgr/filesystem.go
  • Line 139: warning: cyclomatic complexity 21 of function (*Filesystem).writeState() is high (> 15) (gocyclo)

• terraboard/internal/terraform/addrs/parse_ref.go
  • Line 72: warning: cyclomatic complexity 21 of function parseRef() is high (> 15) (gocyclo)

• terraboard/internal/terraform/addrs/parse_target.go
  • Line 27: warning: cyclomatic complexity 18 of function ParseTarget() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/provider_validation.go
  • Line 26: warning: cyclomatic complexity 48 of function validateProviderConfigs() is high (> 15) (gocyclo)

• terraboard/internal/terraform/states/statemgr/testing.go
  • Line 22: warning: cyclomatic complexity 24 of function TestFull() is high (> 15) (gocyclo)

• terraboard/internal/terraform/states/statefile/version3_upgrade.go
  • Line 19: warning: cyclomatic complexity 33 of function upgradeStateV3ToV4() is high (> 15) (gocyclo)
  • Line 252: warning: cyclomatic complexity 21 of function upgradeInstanceObjectV3ToV4() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/configschema/internal_validate.go
  • Line 28: warning: cyclomatic complexity 34 of function (*Block).internalValidate() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/hcl2shim/values_equiv.go
  • Line 24: warning: cyclomatic complexity 21 of function ValuesSDKEquivalent() is high (> 15) (gocyclo)

• terraboard/internal/terraform/configs/resource.go
  • Line 84: warning: cyclomatic complexity 31 of function decodeResourceBlock() is high (> 15) (gocyclo)

golint85%

Golint is a linter for Go source code.

• terraboard/internal/terraform/getproviders/mock_source.go
  • Line 215: warning: exported method MockSource.ForDisplay should have comment or be unexported (golint)

• terraboard/internal/terraform/getproviders/registry_source.go
  • Line 148: warning: exported method RegistrySource.ForDisplay should have comment or be unexported (golint)

• terraboard/internal/terraform/states/remote/state.go
  • Line 41: warning: comment on exported method State.State should be of the form "State ..." (golint)
  • Line 57: warning: comment on exported method State.WriteState should be of the form "WriteState ..." (golint)
  • Line 100: warning: comment on exported method State.RefreshState should be of the form "RefreshState ..." (golint)
  • Line 141: warning: comment on exported method State.PersistState should be of the form "PersistState ..." (golint)

• terraboard/internal/terraform/states/sync.go
  • Line 197: warning: comment on exported method SyncState.SetResourceProvider should be of the form "SetResourceProvider ..." (golint)

• terraboard/internal/terraform/addrs/output_value.go
  • Line 59: warning: exported method AbsOutputValue.Equal should have comment or be unexported (golint)

• terraboard/internal/terraform/addrs/instance_key.go
  • Line 63: warning: exported method IntKey.Value should have comment or be unexported (golint)
  • Line 80: warning: exported method StringKey.Value should have comment or be unexported (golint)
  • Line 132: warning: exported const NoKeyType should have comment (or a comment on this block) or be unexported (golint)

• terraboard/internal/terraform/addrs/module_instance.go
  • Line 29: warning: exported function ParseModuleInstance should have comment or be unexported (golint)

• terraboard/internal/terraform/modsdir/manifest.go
  • Line 51: warning: exported method Manifest.ModuleKey should have comment or be unexported (golint)
  • Line 66: warning: exported function ReadManifestSnapshot should have comment or be unexported (golint)
  • Line 107: warning: exported function ReadManifestSnapshotForDir should have comment or be unexported (golint)
  • Line 119: warning: exported method Manifest.WriteSnapshot should have comment or be unexported (golint)
  • Line 146: warning: exported method Manifest.WriteSnapshotToDir should have comment or be unexported (golint)

• terraboard/internal/terraform/getproviders/registry_client.go
  • Line 58: warning: exported var SupportedPluginProtocols should have comment or be unexported (golint)

• terraboard/internal/terraform/registry/regsrc/friendly_host.go
  • Line 63: warning: exported function NewFriendlyHost should have comment or be unexported (golint)

• terraboard/config/yaml.go
  • Line 9: warning: exported method S3BucketConfig.UnmarshalYAML should have comment or be unexported (golint)
  • Line 22: warning: exported method GitlabConfig.UnmarshalYAML should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/hcl2shim/paths_test.go
  • Line 222: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• terraboard/internal/terraform/states/statefile/file.go
  • Line 34: warning: exported function New should have comment or be unexported (golint)

• terraboard/internal/terraform/httpclient/useragent.go
  • Line 16: warning: comment on exported function UserAgentString should be of the form "UserAgentString ..." (golint)
  • Line 44: warning: exported function TerraformUserAgent should have comment or be unexported (golint)

• terraboard/internal/terraform/tfdiags/diagnostic.go
  • Line 7: warning: exported type Diagnostic should have comment or be unexported (golint)
  • Line 18: warning: exported type Severity should have comment or be unexported (golint)
  • Line 23: warning: exported const Error should have comment (or a comment on this block) or be unexported (golint)
  • Line 27: warning: exported type Description should have comment or be unexported (golint)
  • Line 33: warning: exported type Source should have comment or be unexported (golint)
  • Line 38: warning: exported type FromExpr should have comment or be unexported (golint)

• terraboard/internal/terraform/tfdiags/rpc_friendly.go
  • Line 8: warning: don't use underscores in Go names; struct field Severity_ should be Severity (golint)
  • Line 9: warning: don't use underscores in Go names; struct field Summary_ should be Summary (golint)
  • Line 10: warning: don't use underscores in Go names; struct field Detail_ should be Detail (golint)
  • Line 11: warning: don't use underscores in Go names; struct field Subject_ should be Subject (golint)
  • Line 12: warning: don't use underscores in Go names; struct field Context_ should be Context (golint)

• terraboard/internal/terraform/tfdiags/source_range.go
  • Line 9: warning: exported type SourceRange should have comment or be unexported (golint)
  • Line 14: warning: exported type SourcePos should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/hcl2shim/single_attr_body.go
  • Line 21: warning: exported method SingleAttrBody.Content should have comment or be unexported (golint)
  • Line 36: warning: exported method SingleAttrBody.PartialContent should have comment or be unexported (golint)
  • Line 72: warning: exported method SingleAttrBody.JustAttributes should have comment or be unexported (golint)
  • Line 83: warning: exported method SingleAttrBody.MissingItemRange should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/configschema/empty_value.go
  • Line 46: 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)
  • Line 52: 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)

• terraboard/internal/terraform/states/resource.go
  • Line 193: warning: exported method DeposedKey.GoString should have comment or be unexported (golint)

• terraboard/internal/terraform/states/statemgr/lock.go
  • Line 14: warning: exported method LockDisabled.State should have comment or be unexported (golint)
  • Line 18: warning: exported method LockDisabled.WriteState should have comment or be unexported (golint)
  • Line 22: warning: exported method LockDisabled.RefreshState should have comment or be unexported (golint)
  • Line 26: warning: exported method LockDisabled.PersistState should have comment or be unexported (golint)
  • Line 30: warning: exported method LockDisabled.Lock should have comment or be unexported (golint)
  • Line 34: warning: exported method LockDisabled.Unlock should have comment or be unexported (golint)

• terraboard/internal/terraform/getproviders/errors.go
  • Line 213: warning: comment on exported type ErrRequestCanceled should be of the form "ErrRequestCanceled ..." (with optional leading article) (golint)

• terraboard/internal/terraform/getproviders/package_authentication_test.go
  • Line 59: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• terraboard/internal/terraform/logging/panic.go
  • Line 38: warning: comment on exported function PanicHandler should be of the form "PanicHandler ..." (golint)

• terraboard/internal/terraform/getproviders/filesystem_mirror_source.go
  • Line 126: warning: exported method FilesystemMirrorSource.ForDisplay should have comment or be unexported (golint)

• terraboard/internal/terraform/states/remote/testing.go
  • Line 47: warning: comment on exported function TestRemoteLocks should be of the form "TestRemoteLocks ..." (golint)

• terraboard/internal/terraform/addrs/resource.go
  • Line 31: warning: exported method Resource.Equal should have comment or be unexported (golint)
  • Line 82: warning: exported method ResourceInstance.ContainingResource should have comment or be unexported (golint)
  • Line 93: warning: exported method ResourceInstance.Equal should have comment or be unexported (golint)
  • Line 173: warning: exported method AbsResource.Equal should have comment or be unexported (golint)
  • Line 238: warning: exported method AbsResourceInstance.Equal should have comment or be unexported (golint)
  • Line 322: warning: exported method ConfigResource.Equal should have comment or be unexported (golint)

• terraboard/internal/terraform/modsdir/paths.go
  • Line 3: warning: exported const ManifestSnapshotFilename should have comment or be unexported (golint)

• terraboard/internal/terraform/addrs/provider_config_test.go
  • Line 143: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• terraboard/internal/terraform/configs/provisioner.go
  • Line 209: warning: exported const ProvisionerWhenInvalid should have comment (or a comment on this block) or be unexported (golint)
  • Line 221: warning: exported const ProvisionerOnFailureInvalid should have comment (or a comment on this block) or be unexported (golint)

• terraboard/internal/terraform/logging/logging.go
  • Line 54: warning: comment on exported function RegisterSink should be of the form "RegisterSink ..." (golint)

• terraboard/internal/terraform/getproviders/multi_source.go
  • Line 69: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 241: warning: exported method MultiSource.ForDisplay should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/provider_test.go
  • Line 85: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 139: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• terraboard/internal/terraform/configs/resource.go
  • Line 416: warning: exported type ProviderConfigRef should have comment or be unexported (golint)

• terraboard/internal/terraform/registry/regsrc/module.go
  • Line 13: warning: exported var ErrInvalidModuleSource should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/hcl2shim/flatmap_test.go
  • Line 741: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• terraboard/internal/terraform/addrs/module.go
  • Line 43: warning: exported method Module.Equal should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/module.go
  • Line 505: 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)

• terraboard/internal/terraform/configs/provider_requirements.go
  • Line 25: warning: exported type RequiredProviders should have comment or be unexported (golint)

• terraboard/internal/terraform/getproviders/memoize_source.go
  • Line 101: warning: exported method MemoizeSource.ForDisplay should have comment or be unexported (golint)

• terraboard/internal/terraform/configs/configschema/schema.go
  • Line 7: warning: exported type StringKind should have comment or be unexported (golint)
  • Line 10: warning: exported const StringPlain should have comment (or a comment on this block) or be unexported (golint)

• terraboard/types/db.go
  • Line 36: warning: exported type Lineage should have comment or be unexported (golint)
  • Line 238: warning: exported type PlanStateResourceAttribute should have comment or be unexported (golint)
  • Line 291: warning: exported type PlanOutput should have comment or be unexported (golint)

• terraboard/internal/terraform/addrs/provider_config.go
  • Line 218: warning: exported function ParseLegacyAbsProviderConfigStr should have comment or be unexported (golint)
  • Line 371: warning: comment on exported method AbsProviderConfig.LegacyString should be of the form "LegacyString ..." (golint)
  • Line 376: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell92%

Misspell Finds commonly misspelled English words

• terraboard/internal/terraform/configs/configschema/schema.go
  • Line 138: warning: "additonally" is a misspelling of "additionally" (misspell)

• terraboard/internal/terraform/addrs/provider_config.go
  • Line 33: warning: "absense" is a misspelling of "absence" (misspell)
  • Line 318: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 327: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 344: warning: "inheritence" is a misspelling of "inheritance" (misspell)

• terraboard/internal/terraform/states/statefile/version4.go
  • Line 331: warning: "occured" is a misspelling of "occurred" (misspell)
  • Line 341: warning: "occured" is a misspelling of "occurred" (misspell)
  • Line 412: warning: "occured" is a misspelling of "occurred" (misspell)
  • Line 423: warning: "occured" is a misspelling of "occurred" (misspell)

• terraboard/internal/terraform/getproviders/http_mirror_source.go
  • Line 413: warning: "absense" is a misspelling of "absence" (misspell)

• terraboard/internal/terraform/getproviders/types.go
  • Line 18: warning: "absense" is a misspelling of "absence" (misspell)
  • Line 172: warning: "receving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/configs/backend.go
  • Line 29: warning: "reciever" is a misspelling of "receiver" (misspell)

• terraboard/internal/terraform/configs/module_merge.go
  • Line 68: warning: "interchangable" is a misspelling of "interchangeable" (misspell)

• terraboard/internal/terraform/states/module.go
  • Line 304: warning: "receving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/configs/configschema/empty_value.go
  • Line 7: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 12: warning: "recieving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/addrs/resource_phase.go
  • Line 10: warning: "accomodate" is a misspelling of "accommodate" (misspell)
  • Line 23: warning: "receving" is a misspelling of "receiving" (misspell)
  • Line 70: warning: "accomodate" is a misspelling of "accommodate" (misspell)
  • Line 89: warning: "receving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/getproviders/hash.go
  • Line 66: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 84: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 399: warning: "reciever" is a misspelling of "receiver" (misspell)
  • Line 432: warning: "signular" is a misspelling of "singular" (misspell)

• terraboard/internal/terraform/states/state.go
  • Line 21: warning: "acheive" is a misspelling of "achieve" (misspell)

• terraboard/internal/terraform/states/state_equal.go
  • Line 23: warning: "reciever" is a misspelling of "receiver" (misspell)

• terraboard/internal/terraform/addrs/module_instance.go
  • Line 393: warning: "reciever" is a misspelling of "receiver" (misspell)

• terraboard/internal/terraform/addrs/resource.go
  • Line 201: warning: "receving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/getproviders/errors.go
  • Line 179: warning: "absense" is a misspelling of "absence" (misspell)

• terraboard/internal/terraform/getproviders/multi_source.go
  • Line 197: warning: "absense" is a misspelling of "absence" (misspell)

• terraboard/internal/terraform/states/resource.go
  • Line 175: warning: "absense" is a misspelling of "absence" (misspell)
  • Line 204: warning: "reciever" is a misspelling of "receiver" (misspell)

• terraboard/internal/terraform/tfdiags/consolidate_warnings.go
  • Line 15: warning: "reciever" is a misspelling of "receiver" (misspell)

• terraboard/internal/terraform/addrs/instance_key.go
  • Line 47: warning: "absense" is a misspelling of "absence" (misspell)

• terraboard/internal/terraform/addrs/module_call.go
  • Line 48: warning: "receving" is a misspelling of "receiving" (misspell)
  • Line 97: warning: "receving" is a misspelling of "receiving" (misspell)

• terraboard/internal/terraform/states/state_deepcopy.go
  • Line 14: warning: "reciever" is a misspelling of "receiver" (misspell)
  • Line 18: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 40: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 75: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 100: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 124: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 183: warning: "recieving" is a misspelling of "receiving" (misspell)
  • Line 219: warning: "recieving" is a misspelling of "receiving" (misspell)