Preparing report...

Report for github.com/mitchellh/packer

A+    Excellent!    Found 156 issues across 317 files

Tweet

gofmt91%

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


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!


gocyclo91%

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.

    • packer/hcl2template/shim/values.go
    • Line 90: warning: cyclomatic complexity 29 of function WriteUnknownPlaceholderValues() is high (> 15) (gocyclo)
    • Line 18: warning: cyclomatic complexity 17 of function ConfigValueFromHCL2() is high (> 15) (gocyclo)
    • packer/command/hcl2_upgrade.go
    • Line 550: warning: cyclomatic complexity 21 of function jsonBodyToHCL2Body() is high (> 15) (gocyclo)
    • Line 133: warning: cyclomatic complexity 17 of function (*HCL2UpgradeCommand).RunContext() is high (> 15) (gocyclo)
    • Line 286: warning: cyclomatic complexity 17 of function transposeTemplatingCalls() is high (> 15) (gocyclo)
    • packer/provisioner/inspec/provisioner.go
    • Line 199: warning: cyclomatic complexity 23 of function (*Provisioner).Provision() is high (> 15) (gocyclo)
    • Line 76: warning: cyclomatic complexity 23 of function (*Provisioner).Prepare() is high (> 15) (gocyclo)
    • Line 350: warning: cyclomatic complexity 19 of function (*Provisioner).executeInspec() is high (> 15) (gocyclo)

golint52%

Golint is a linter for Go source code.

    • packer/fix/fixer_parallels_deprecations.go
    • Line 12: warning: exported method FixerParallelsDeprecations.DeprecatedOptions should have comment or be unexported (golint)
    • Line 18: warning: exported method FixerParallelsDeprecations.Fix should have comment or be unexported (golint)
    • Line 62: warning: exported method FixerParallelsDeprecations.Synopsis should have comment or be unexported (golint)
    • packer/builder/profitbricks/artifact.go
    • Line 7: warning: exported type Artifact should have comment or be unexported (golint)
    • Line 15: warning: exported method Artifact.BuilderId should have comment or be unexported (golint)
    • Line 19: warning: exported method Artifact.Files should have comment or be unexported (golint)
    • Line 23: warning: exported method Artifact.Id should have comment or be unexported (golint)
    • Line 31: warning: exported method Artifact.State should have comment or be unexported (golint)
    • Line 35: warning: exported method Artifact.Destroy should have comment or be unexported (golint)
    • packer/provisioner/inspec/provisioner_test.go
    • Line 27: warning: don't use underscores in Go names; var inspec_stub should be inspecStub (golint)
    • Line 56: warning: don't use underscores in Go names; var hostkey_file should be hostkeyFile (golint)
    • Line 62: warning: don't use underscores in Go names; var publickey_file should be publickeyFile (golint)
    • Line 68: warning: don't use underscores in Go names; var profile_file should be profileFile (golint)
    • Line 98: warning: don't use underscores in Go names; var hostkey_file should be hostkeyFile (golint)
    • Line 104: warning: don't use underscores in Go names; var publickey_file should be publickeyFile (golint)
    • Line 118: warning: don't use underscores in Go names; var profile_file should be profileFile (golint)
    • Line 130: warning: don't use underscores in Go names; var test_dir should be testDir (golint)
    • Line 148: warning: don't use underscores in Go names; var publickey_file should be publickeyFile (golint)
    • Line 154: warning: don't use underscores in Go names; var profile_file should be profileFile (golint)
    • Line 175: warning: don't use underscores in Go names; var hostkey_file should be hostkeyFile (golint)
    • Line 193: warning: don't use underscores in Go names; var hostkey_file should be hostkeyFile (golint)
    • Line 199: warning: don't use underscores in Go names; var profile_file should be profileFile (golint)
    • Line 220: warning: don't use underscores in Go names; var publickey_file should be publickeyFile (golint)
    • Line 238: warning: don't use underscores in Go names; var hostkey_file should be hostkeyFile (golint)
    • Line 244: warning: don't use underscores in Go names; var publickey_file should be publickeyFile (golint)
    • Line 250: warning: don't use underscores in Go names; var profile_file should be profileFile (golint)
    • packer/hcl2template/utils.go
    • Line 118: warning: exported function PrintableCtyValue should have comment or be unexported (golint)
    • Line 127: warning: exported function ConvertPluginConfigValueToHCLValue should have comment or be unexported (golint)
    • packer/builder/null/artifact_export.go
    • Line 7: warning: comment on exported type NullArtifact should be of the form "NullArtifact ..." (with optional leading article) (golint)
    • Line 11: warning: exported method NullArtifact.BuilderId should have comment or be unexported (golint)
    • Line 15: warning: exported method NullArtifact.Files should have comment or be unexported (golint)
    • Line 19: warning: exported method NullArtifact.Id should have comment or be unexported (golint)
    • Line 27: warning: exported method NullArtifact.State should have comment or be unexported (golint)
    • Line 36: warning: exported method NullArtifact.Destroy should have comment or be unexported (golint)
    • packer/packer/testing.go
    • Line 11: warning: exported function TestCoreConfig should have comment or be unexported (golint)
    • Line 26: warning: exported function TestCore should have comment or be unexported (golint)
    • Line 36: warning: exported function TestUi should have comment or be unexported (golint)
    • packer/fix/fixer_hyperv_cpu_and_ram_naming.go
    • Line 10: warning: exported method FizerHypervCPUandRAM.DeprecatedOptions should have comment or be unexported (golint)
    • Line 16: warning: exported method FizerHypervCPUandRAM.Fix should have comment or be unexported (golint)
    • Line 60: warning: exported method FizerHypervCPUandRAM.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_sshkeypath.go
    • Line 11: warning: exported method FixerSSHKeyPath.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerSSHKeyPath.Fix should have comment or be unexported (golint)
    • Line 54: warning: exported method FixerSSHKeyPath.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_vmware_rename.go
    • Line 10: warning: exported method FixerVMwareRename.DeprecatedOptions should have comment or be unexported (golint)
    • Line 14: warning: exported method FixerVMwareRename.Fix should have comment or be unexported (golint)
    • Line 48: warning: exported method FixerVMwareRename.Synopsis should have comment or be unexported (golint)
    • packer/command/fmt.go
    • Line 12: warning: exported type FormatCommand should have comment or be unexported (golint)
    • Line 16: warning: exported method FormatCommand.Run should have comment or be unexported (golint)
    • Line 26: warning: exported method FormatCommand.ParseArgs should have comment or be unexported (golint)
    • Line 45: warning: exported method FormatCommand.RunContext should have comment or be unexported (golint)
    • Line 71: warning: exported method FormatCommand.Help should have comment or be unexported (golint)
    • Line 101: warning: exported method FormatCommand.Synopsis should have comment or be unexported (golint)
    • Line 105: warning: exported method FormatCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 109: warning: exported method FormatCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/hcl2template/types.hcl_post-processor.go
    • Line 24: warning: exported method HCL2PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 28: warning: exported method HCL2PostProcessor.HCL2Prepare should have comment or be unexported (golint)
    • Line 62: warning: exported method HCL2PostProcessor.Configure should have comment or be unexported (golint)
    • Line 66: warning: exported method HCL2PostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/packer/telemetry.go
    • Line 16: warning: exported const TelemetryVersion should have comment or be unexported (golint)
    • Line 17: warning: exported const TelemetryPanicVersion should have comment or be unexported (golint)
    • Line 19: warning: exported var CheckpointReporter should have comment or be unexported (golint)
    • Line 21: warning: exported type PackerReport should have comment or be unexported (golint)
    • Line 28: warning: exported type CheckpointTelemetry should have comment or be unexported (golint)
    • Line 34: warning: exported function NewCheckpointReporter should have comment or be unexported (golint)
    • Line 74: warning: exported method CheckpointTelemetry.ReportPanic should have comment or be unexported (golint)
    • Line 90: warning: exported method CheckpointTelemetry.AddSpan should have comment or be unexported (golint)
    • Line 106: warning: exported method CheckpointTelemetry.Finalize should have comment or be unexported (golint)
    • Line 133: warning: exported type TelemetrySpan should have comment or be unexported (golint)
    • Line 142: warning: exported method TelemetrySpan.End should have comment or be unexported (golint)
    • packer/hcl2template/types.variables.go
    • Line 21: warning: comment on exported type LocalBlock should be of the form "LocalBlock ..." (with optional leading article) (golint)
    • Line 42: warning: exported type Variable should have comment or be unexported (golint)
    • Line 73: warning: exported method Variable.GoString should have comment or be unexported (golint)
    • Line 180: warning: exported type Variables should have comment or be unexported (golint)
    • Line 182: warning: exported method Variables.Keys should have comment or be unexported (golint)
    • Line 190: warning: exported method Variables.Values should have comment or be unexported (golint)
    • Line 199: warning: exported method Variables.ValidateValues should have comment or be unexported (golint)
    • Line 564: warning: comment on exported const VarEnvPrefix should be of the form "VarEnvPrefix ..." (golint)
    • packer/command/core_wrapper.go
    • Line 15: warning: exported method CoreWrapper.Initialize should have comment or be unexported (golint)
    • Line 28: warning: exported method CoreWrapper.PluginRequirements should have comment or be unexported (golint)
    • packer/command/init.go
    • Line 19: warning: exported type InitCommand should have comment or be unexported (golint)
    • Line 23: warning: exported method InitCommand.Run should have comment or be unexported (golint)
    • Line 35: warning: exported method InitCommand.ParseArgs should have comment or be unexported (golint)
    • Line 53: warning: exported method InitCommand.RunContext should have comment or be unexported (golint)
    • Line 177: warning: exported method InitCommand.Help should have comment or be unexported (golint)
    • Line 201: warning: exported method InitCommand.Synopsis should have comment or be unexported (golint)
    • Line 205: warning: exported method InitCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 209: warning: exported method InitCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/log.go
    • Line 12: warning: comment on exported const EnvLog should be of the form "EnvLog ..." (golint)
    • Line 15: warning: exported const EnvLogFile should have comment or be unexported (golint)
    • Line 70: warning: exported function ScanLinesSmallerThanBuffer should have comment or be unexported (golint)
    • packer/builder/file/builder.go
    • Line 21: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 23: warning: exported type Builder should have comment or be unexported (golint)
    • Line 28: warning: exported method Builder.ConfigSpec should have comment or be unexported (golint)
    • Line 30: warning: exported method Builder.Prepare should have comment or be unexported (golint)
    • packer/command/build.go
    • Line 27: warning: exported type BuildCommand should have comment or be unexported (golint)
    • Line 31: warning: exported method BuildCommand.Run should have comment or be unexported (golint)
    • Line 43: warning: exported method BuildCommand.ParseArgs should have comment or be unexported (golint)
    • Line 65: warning: exported method Meta.GetConfigFromHCL should have comment or be unexported (golint)
    • Line 94: warning: exported method Meta.GetConfig should have comment or be unexported (golint)
    • Line 116: warning: exported method Meta.GetConfigFromJSON should have comment or be unexported (golint)
    • Line 147: warning: exported method BuildCommand.RunContext should have comment or be unexported (golint)
    • Line 380: warning: exported method BuildCommand.Help should have comment or be unexported (golint)
    • Line 405: warning: exported method BuildCommand.Synopsis should have comment or be unexported (golint)
    • Line 409: warning: exported method BuildCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 413: warning: exported method BuildCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/post-processor/compress/artifact.go
    • Line 8: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 10: warning: exported type Artifact should have comment or be unexported (golint)
    • Line 14: warning: exported method Artifact.BuilderId should have comment or be unexported (golint)
    • Line 18: warning: exported method Artifact.Id should have comment or be unexported (golint)
    • Line 22: warning: exported method Artifact.Files should have comment or be unexported (golint)
    • Line 30: warning: exported method Artifact.State should have comment or be unexported (golint)
    • Line 34: warning: exported method Artifact.Destroy should have comment or be unexported (golint)
    • packer/fix/fixer_proxmox_type.go
    • Line 10: warning: exported method FixerProxmoxType.DeprecatedOptions should have comment or be unexported (golint)
    • Line 14: warning: exported method FixerProxmoxType.Fix should have comment or be unexported (golint)
    • Line 47: warning: exported method FixerProxmoxType.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_pp_docker_tag_tags.go
    • Line 12: warning: exported method FixerDockerTagtoTags.DeprecatedOptions should have comment or be unexported (golint)
    • Line 18: warning: exported method FixerDockerTagtoTags.Fix should have comment or be unexported (golint)
    • Line 110: warning: exported method FixerDockerTagtoTags.Synopsis should have comment or be unexported (golint)
    • packer/provisioner/salt-masterless/provisioner.go
    • Line 3: warning: package comment should be of the form "Package saltmasterless ..." (golint)
    • Line 26: warning: exported type Config should have comment or be unexported (golint)
    • Line 83: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 117: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 119: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 190: warning: don't use underscores in Go names; var cmd_args should be cmdArgs (golint)
    • Line 242: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • Line 524: warning: don't use underscores in Go names; var temp_dst should be tempDst (golint)
    • Line 601: warning: don't use underscores in Go names; var temp_dst should be tempDst (golint)
    • packer/post-processor/shell-local/post-processor.go
    • Line 1: warning: don't use an underscore in package name (golint)
    • Line 11: warning: exported type PostProcessor should have comment or be unexported (golint)
    • Line 15: warning: exported type ExecuteCommandTemplate should have comment or be unexported (golint)
    • Line 20: warning: exported method PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 22: warning: exported method PostProcessor.Configure should have comment or be unexported (golint)
    • Line 43: warning: exported method PostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/fix/fixer_amazon_temporary_security_group_cidrs.go
    • Line 9: warning: exported type FixerAmazonTemporarySecurityCIDRs should have comment or be unexported (golint)
    • Line 11: warning: exported method FixerAmazonTemporarySecurityCIDRs.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerAmazonTemporarySecurityCIDRs.Fix should have comment or be unexported (golint)
    • Line 58: warning: exported method FixerAmazonTemporarySecurityCIDRs.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_hyperv_deprecations.go
    • Line 11: warning: exported method FixerHypervDeprecations.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerHypervDeprecations.Fix should have comment or be unexported (golint)
    • Line 54: warning: exported method FixerHypervDeprecations.Synopsis should have comment or be unexported (golint)
    • packer/packer/progressbar.go
    • Line 13: warning: exported function ProgressBarConfig should have comment or be unexported (golint)
    • Line 26: warning: exported method UiProgressBar.TrackProgress should have comment or be unexported (golint)
    • packer/acctest/testing.go
    • Line 78: warning: exported type TestBuilderSet should have comment or be unexported (golint)
    • Line 83: warning: exported method TestBuilderSet.Start should have comment or be unexported (golint)
    • packer/fix/fixer_hyperv_vmxc_typo.go
    • Line 11: warning: exported method FixerHypervVmxcTypo.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerHypervVmxcTypo.Fix should have comment or be unexported (golint)
    • Line 55: warning: exported method FixerHypervVmxcTypo.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_parallels_headless.go
    • Line 10: warning: exported method FixerParallelsHeadless.DeprecatedOptions should have comment or be unexported (golint)
    • Line 16: warning: exported method FixerParallelsHeadless.Fix should have comment or be unexported (golint)
    • Line 55: warning: exported method FixerParallelsHeadless.Synopsis should have comment or be unexported (golint)
    • packer/packer/cmd_post_processor.go
    • Line 25: warning: receiver name c should be consistent with previous receiver name b for cmdPostProcessor (golint)
    • Line 34: warning: receiver name c should be consistent with previous receiver name b for cmdPostProcessor (golint)
    • Line 43: warning: receiver name c should be consistent with previous receiver name b for cmdPostProcessor (golint)
    • packer/packer/ui.go
    • Line 20: warning: exported var ErrInterrupted should have comment or be unexported (golint)
    • Line 22: warning: exported type UiColor should have comment or be unexported (golint)
    • Line 25: warning: exported const UiColorRed should have comment (or a comment on this block) or be unexported (golint)
    • Line 43: warning: exported method ColoredUi.Ask should have comment or be unexported (golint)
    • Line 47: warning: exported method ColoredUi.Say should have comment or be unexported (golint)
    • Line 51: warning: exported method ColoredUi.Message should have comment or be unexported (golint)
    • Line 64: warning: exported method ColoredUi.Machine should have comment or be unexported (golint)
    • Line 69: warning: exported method ColoredUi.TrackProgress should have comment or be unexported (golint)
    • Line 117: warning: exported method TargetedUI.Ask should have comment or be unexported (golint)
    • Line 121: warning: exported method TargetedUI.Say should have comment or be unexported (golint)
    • Line 125: warning: exported method TargetedUI.Message should have comment or be unexported (golint)
    • Line 133: warning: exported method TargetedUI.Machine should have comment or be unexported (golint)
    • Line 153: warning: exported method TargetedUI.TrackProgress should have comment or be unexported (golint)
    • Line 166: warning: exported method MachineReadableUi.Ask should have comment or be unexported (golint)
    • Line 170: warning: exported method MachineReadableUi.Say should have comment or be unexported (golint)
    • Line 174: warning: exported method MachineReadableUi.Message should have comment or be unexported (golint)
    • Line 182: warning: exported method MachineReadableUi.Machine should have comment or be unexported (golint)
    • Line 215: warning: exported method MachineReadableUi.TrackProgress should have comment or be unexported (golint)
    • Line 228: warning: exported method TimestampedUi.Ask should have comment or be unexported (golint)
    • Line 232: warning: exported method TimestampedUi.Say should have comment or be unexported (golint)
    • Line 236: warning: exported method TimestampedUi.Message should have comment or be unexported (golint)
    • Line 244: warning: exported method TimestampedUi.Machine should have comment or be unexported (golint)
    • Line 248: warning: exported method TimestampedUi.TrackProgress should have comment or be unexported (golint)
    • packer/provisioner/file/provisioner.go
    • Line 22: warning: exported type Config should have comment or be unexported (golint)
    • Line 63: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 67: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 69: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 122: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • Line 130: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 135: warning: exported method Provisioner.ProvisionDownload should have comment or be unexported (golint)
    • Line 186: warning: exported method Provisioner.ProvisionUpload should have comment or be unexported (golint)
    • packer/fix/fixer_amazon_spot_price_product.go
    • Line 11: warning: exported method FixerAmazonSpotPriceProductDeprecation.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerAmazonSpotPriceProductDeprecation.Fix should have comment or be unexported (golint)
    • Line 64: warning: exported method FixerAmazonSpotPriceProductDeprecation.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_createtime.go
    • Line 13: warning: exported method FixerCreateTime.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerCreateTime.Fix should have comment or be unexported (golint)
    • Line 58: warning: exported method FixerCreateTime.Synopsis should have comment or be unexported (golint)
    • packer/hcl2template/internal/mock.go
    • Line 17: warning: exported type NestedMockConfig should have comment or be unexported (golint)
    • Line 33: warning: exported type MockTag should have comment or be unexported (golint)
    • Line 38: warning: exported type MockConfig should have comment or be unexported (golint)
    • Line 45: warning: exported method MockConfig.Prepare should have comment or be unexported (golint)
    • Line 70: warning: exported type MockBuilder should have comment or be unexported (golint)
    • Line 76: warning: exported method MockBuilder.ConfigSpec should have comment or be unexported (golint)
    • Line 78: warning: exported method MockBuilder.Prepare should have comment or be unexported (golint)
    • Line 82: warning: exported method MockBuilder.Run should have comment or be unexported (golint)
    • Line 90: warning: exported type MockProvisioner should have comment or be unexported (golint)
    • Line 96: warning: exported method MockProvisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 100: warning: exported method MockProvisioner.Prepare should have comment or be unexported (golint)
    • Line 104: warning: exported method MockProvisioner.Provision should have comment or be unexported (golint)
    • Line 112: warning: exported type MockDatasource should have comment or be unexported (golint)
    • Line 118: warning: exported method MockDatasource.ConfigSpec should have comment or be unexported (golint)
    • Line 122: warning: exported method MockDatasource.OutputSpec should have comment or be unexported (golint)
    • Line 126: warning: exported method MockDatasource.Configure should have comment or be unexported (golint)
    • Line 130: warning: exported method MockDatasource.Execute should have comment or be unexported (golint)
    • Line 138: warning: exported type MockPostProcessor should have comment or be unexported (golint)
    • Line 144: warning: exported method MockPostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 148: warning: exported method MockPostProcessor.Configure should have comment or be unexported (golint)
    • Line 152: warning: exported method MockPostProcessor.PostProcess should have comment or be unexported (golint)
    • Line 160: warning: exported type MockCommunicator should have comment or be unexported (golint)
    • Line 167: warning: exported method MockCommunicator.ConfigSpec should have comment or be unexported (golint)
    • Line 171: warning: exported method MockCommunicator.Configure should have comment or be unexported (golint)
    • Line 179: warning: exported type NamedMapStringString should have comment or be unexported (golint)
    • Line 180: warning: exported type NamedString should have comment or be unexported (golint)
    • packer/fix/fixer_amazon_shutdown_behavior.go
    • Line 13: warning: exported method FixerAmazonShutdownBehavior.DeprecatedOptions should have comment or be unexported (golint)
    • Line 19: warning: exported method FixerAmazonShutdownBehavior.Fix should have comment or be unexported (golint)
    • Line 58: warning: exported method FixerAmazonShutdownBehavior.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_qemu_disk_size.go
    • Line 12: warning: exported method FixerQEMUDiskSize.DeprecatedOptions should have comment or be unexported (golint)
    • Line 16: warning: exported method FixerQEMUDiskSize.Fix should have comment or be unexported (golint)
    • Line 55: warning: exported method FixerQEMUDiskSize.Synopsis should have comment or be unexported (golint)
    • packer/version/version.go
    • Line 8: warning: comment on exported var GitCommit should be of the form "GitCommit ..." (golint)
    • Line 11: warning: comment on exported const Version should be of the form "Version ..." (golint)
    • Line 14: warning: comment on exported const VersionPrerelease should be of the form "VersionPrerelease ..." (golint)
    • Line 19: warning: exported var PackerVersion should have comment or be unexported (golint)
    • Line 21: warning: exported function FormattedVersion should have comment or be unexported (golint)
    • packer/fix/fixer_iso_md5.go
    • Line 11: warning: exported method FixerISOMD5.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerISOMD5.Fix should have comment or be unexported (golint)
    • Line 51: warning: exported method FixerISOMD5.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_ssh_timeout.go
    • Line 10: warning: exported method FixerSSHTimout.DeprecatedOptions should have comment or be unexported (golint)
    • Line 16: warning: exported method FixerSSHTimout.Fix should have comment or be unexported (golint)
    • Line 69: warning: exported method FixerSSHTimout.Synopsis should have comment or be unexported (golint)
    • packer/packer/provisioner.go
    • Line 22: warning: comment on exported type ProvisionHook should be of the form "ProvisionHook ..." (with optional leading article) (golint)
    • Line 54: warning: comment on exported function BasicPlaceholderData should be of the form "BasicPlaceholderData ..." (golint)
    • Line 76: warning: exported function CastDataToMap should have comment or be unexported (golint)
    • Line 105: warning: comment on exported method ProvisionHook.Run should be of the form "Run ..." (golint)
    • Line 140: warning: exported method PausedProvisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 141: warning: exported method PausedProvisioner.FlatConfig should have comment or be unexported (golint)
    • Line 142: warning: exported method PausedProvisioner.Prepare should have comment or be unexported (golint)
    • Line 146: warning: exported method PausedProvisioner.Provision should have comment or be unexported (golint)
    • Line 166: warning: exported method RetriedProvisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 167: warning: exported method RetriedProvisioner.FlatConfig should have comment or be unexported (golint)
    • Line 168: warning: exported method RetriedProvisioner.Prepare should have comment or be unexported (golint)
    • Line 172: warning: exported method RetriedProvisioner.Provision should have comment or be unexported (golint)
    • Line 211: warning: exported method DebuggedProvisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 212: warning: exported method DebuggedProvisioner.FlatConfig should have comment or be unexported (golint)
    • Line 213: warning: exported method DebuggedProvisioner.Prepare should have comment or be unexported (golint)
    • Line 217: warning: exported method DebuggedProvisioner.Provision should have comment or be unexported (golint)
    • packer/command/validate.go
    • Line 12: warning: exported type ValidateCommand should have comment or be unexported (golint)
    • Line 16: warning: exported method ValidateCommand.Run should have comment or be unexported (golint)
    • Line 28: warning: exported method ValidateCommand.ParseArgs should have comment or be unexported (golint)
    • Line 47: warning: exported method ValidateCommand.RunContext should have comment or be unexported (golint)
    • Line 80: warning: exported method ValidateCommand.Help should have comment or be unexported (golint)
    • Line 104: warning: exported method ValidateCommand.Synopsis should have comment or be unexported (golint)
    • Line 108: warning: exported method ValidateCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 112: warning: exported method ValidateCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/packer/run_interfaces.go
    • Line 9: warning: exported type GetBuildsOptions should have comment or be unexported (golint)
    • Line 17: warning: exported type BuildGetter should have comment or be unexported (golint)
    • Line 24: warning: exported type Evaluator should have comment or be unexported (golint)
    • Line 31: warning: exported type InitializeOptions should have comment or be unexported (golint)
    • Line 52: warning: comment on exported type FixConfigMode should be of the form "FixConfigMode ..." (with optional leading article) (golint)
    • Line 65: warning: exported type FixConfigOptions should have comment or be unexported (golint)
    • Line 69: warning: exported type ConfigFixer should have comment or be unexported (golint)
    • Line 74: warning: exported type InspectConfigOptions should have comment or be unexported (golint)
    • Line 78: warning: exported type ConfigInspector should have comment or be unexported (golint)
    • packer/command/console.go
    • Line 17: warning: exported var TiniestBuilder should have comment or be unexported (golint)
    • Line 26: warning: exported type ConsoleCommand should have comment or be unexported (golint)
    • Line 30: warning: exported method ConsoleCommand.Run should have comment or be unexported (golint)
    • Line 41: warning: exported method ConsoleCommand.ParseArgs should have comment or be unexported (golint)
    • Line 57: warning: exported method ConsoleCommand.RunContext should have comment or be unexported (golint)
    • Line 73: warning: exported method ConsoleCommand.Help should have comment or be unexported (golint)
    • Line 90: warning: exported method ConsoleCommand.Synopsis should have comment or be unexported (golint)
    • Line 94: warning: exported method ConsoleCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 98: warning: exported method ConsoleCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/fix/fixer_docker_email.go
    • Line 5: warning: exported type FixerDockerEmail should have comment or be unexported (golint)
    • Line 7: warning: exported method FixerDockerEmail.DeprecatedOptions should have comment or be unexported (golint)
    • Line 15: warning: exported method FixerDockerEmail.Fix should have comment or be unexported (golint)
    • Line 57: warning: exported method FixerDockerEmail.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_qemu_host_port.go
    • Line 10: warning: exported method FixerQEMUHostPort.Fix should have comment or be unexported (golint)
    • Line 55: warning: exported method FixerQEMUHostPort.Synopsis should have comment or be unexported (golint)
    • Line 59: warning: exported method FixerQEMUHostPort.DeprecatedOptions should have comment or be unexported (golint)
    • packer/packer/maps.go
    • Line 9: warning: exported type MapOfProvisioner should have comment or be unexported (golint)
    • Line 11: warning: exported method MapOfProvisioner.Has should have comment or be unexported (golint)
    • Line 16: warning: exported method MapOfProvisioner.Set should have comment or be unexported (golint)
    • Line 20: warning: exported method MapOfProvisioner.Start should have comment or be unexported (golint)
    • Line 28: warning: exported method MapOfProvisioner.List should have comment or be unexported (golint)
    • Line 36: warning: exported type MapOfPostProcessor should have comment or be unexported (golint)
    • Line 38: warning: exported method MapOfPostProcessor.Has should have comment or be unexported (golint)
    • Line 43: warning: exported method MapOfPostProcessor.Set should have comment or be unexported (golint)
    • Line 47: warning: exported method MapOfPostProcessor.Start should have comment or be unexported (golint)
    • Line 55: warning: exported method MapOfPostProcessor.List should have comment or be unexported (golint)
    • Line 63: warning: exported type MapOfBuilder should have comment or be unexported (golint)
    • Line 65: warning: exported method MapOfBuilder.Has should have comment or be unexported (golint)
    • Line 70: warning: exported method MapOfBuilder.Set should have comment or be unexported (golint)
    • Line 74: warning: exported method MapOfBuilder.Start should have comment or be unexported (golint)
    • Line 82: warning: exported method MapOfBuilder.List should have comment or be unexported (golint)
    • Line 90: warning: exported type MapOfDatasource should have comment or be unexported (golint)
    • Line 92: warning: exported method MapOfDatasource.Has should have comment or be unexported (golint)
    • Line 97: warning: exported method MapOfDatasource.Set should have comment or be unexported (golint)
    • Line 101: warning: exported method MapOfDatasource.Start should have comment or be unexported (golint)
    • Line 109: warning: exported method MapOfDatasource.List should have comment or be unexported (golint)
    • packer/fix/fixer_vmware_compaction.go
    • Line 10: warning: exported method FixerVMwareCompaction.DeprecatedOptions should have comment or be unexported (golint)
    • Line 14: warning: exported method FixerVMwareCompaction.Fix should have comment or be unexported (golint)
    • Line 93: warning: exported method FixerVMwareCompaction.Synopsis should have comment or be unexported (golint)
    • packer/command/inspect.go
    • Line 11: warning: exported type InspectCommand should have comment or be unexported (golint)
    • Line 15: warning: exported method InspectCommand.Run should have comment or be unexported (golint)
    • Line 26: warning: exported method InspectCommand.ParseArgs should have comment or be unexported (golint)
    • Line 42: warning: exported method InspectCommand.RunContext should have comment or be unexported (golint)
    • Line 56: warning: exported method InspectCommand.Help should have comment or be unexported (golint)
    • Line 72: warning: exported method InspectCommand.Synopsis should have comment or be unexported (golint)
    • Line 76: warning: exported method InspectCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 80: warning: exported method InspectCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/provisioner/shell-local/provisioner.go
    • Line 11: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 15: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 17: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 31: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/fix/fixer_amazon_enhanced_networking.go
    • Line 13: warning: exported method FixerAmazonEnhancedNetworking.DeprecatedOptions should have comment or be unexported (golint)
    • Line 19: warning: exported method FixerAmazonEnhancedNetworking.Fix should have comment or be unexported (golint)
    • Line 64: warning: exported method FixerAmazonEnhancedNetworking.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_clean_image_name.go
    • Line 14: warning: exported method FixerCleanImageName.DeprecatedOptions should have comment or be unexported (golint)
    • Line 22: warning: exported method FixerCleanImageName.Fix should have comment or be unexported (golint)
    • Line 68: warning: exported method FixerCleanImageName.Synopsis should have comment or be unexported (golint)
    • packer/builder/profitbricks/step_create_ssh_key.go
    • Line 14: warning: exported type StepCreateSSHKey should have comment or be unexported (golint)
    • Line 19: warning: exported method StepCreateSSHKey.Run should have comment or be unexported (golint)
    • Line 42: warning: don't use underscores in Go names; var priv_blk should be privBlk (golint)
    • Line 61: warning: exported method StepCreateSSHKey.Cleanup should have comment or be unexported (golint)
    • packer/fix/fixer_comm_config.go
    • Line 13: warning: exported method FixerCommConfig.DeprecatedOptions should have comment or be unexported (golint)
    • Line 20: warning: exported method FixerCommConfig.Fix should have comment or be unexported (golint)
    • Line 107: warning: exported method FixerCommConfig.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_sshdisableagent.go
    • Line 11: warning: exported method FixerSSHDisableAgent.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerSSHDisableAgent.Fix should have comment or be unexported (golint)
    • Line 54: warning: exported method FixerSSHDisableAgent.Synopsis should have comment or be unexported (golint)
    • packer/builder/file/artifact.go
    • Line 9: warning: exported type FileArtifact should have comment or be unexported (golint)
    • Line 13: warning: exported method FileArtifact.BuilderId should have comment or be unexported (golint)
    • Line 17: warning: exported method FileArtifact.Files should have comment or be unexported (golint)
    • Line 21: warning: exported method FileArtifact.Id should have comment or be unexported (golint)
    • Line 29: warning: exported method FileArtifact.State should have comment or be unexported (golint)
    • Line 33: warning: exported method FileArtifact.Destroy should have comment or be unexported (golint)
    • packer/provisioner/powershell/provisioner.go
    • Line 3: warning: package comment should be of the form "Package powershell ..." (golint)
    • Line 40: warning: exported type Config should have comment or be unexported (golint)
    • Line 93: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 116: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 118: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 260: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • Line 480: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 533: warning: exported method Provisioner.Communicator should have comment or be unexported (golint)
    • Line 537: warning: exported method Provisioner.ElevatedUser should have comment or be unexported (golint)
    • Line 541: warning: exported method Provisioner.ElevatedPassword should have comment or be unexported (golint)
    • packer/fix/fixer_pp_vagrant_override.go
    • Line 10: warning: exported method FixerVagrantPPOverride.DeprecatedOptions should have comment or be unexported (golint)
    • Line 14: warning: exported method FixerVagrantPPOverride.Fix should have comment or be unexported (golint)
    • Line 65: warning: exported method FixerVagrantPPOverride.Synopsis should have comment or be unexported (golint)
    • packer/commands.go
    • Line 15: warning: exported const ErrorPrefix should have comment or be unexported (golint)
    • Line 16: warning: exported const OutputPrefix should have comment or be unexported (golint)
    • packer/post-processor/artifice/post-processor.go
    • Line 23: warning: exported type Config should have comment or be unexported (golint)
    • Line 32: warning: exported type PostProcessor should have comment or be unexported (golint)
    • Line 36: warning: exported method PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 38: warning: exported method PostProcessor.Configure should have comment or be unexported (golint)
    • Line 58: warning: exported method PostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/packer/plugin-getter/plugins.go
    • Line 22: warning: exported type Requirements should have comment or be unexported (golint)
    • Line 46: warning: exported type BinaryInstallationOptions should have comment or be unexported (golint)
    • Line 59: warning: exported type ListInstallationsOptions should have comment or be unexported (golint)
    • Line 67: warning: exported method Requirement.FilenamePrefix should have comment or be unexported (golint)
    • Line 219: warning: exported type GetOptions should have comment or be unexported (golint)
    • Line 235: warning: exported method BinaryInstallationOptions.CheckProtocolVersion should have comment or be unexported (golint)
    • Line 235: warning: receiver name binOpts should be consistent with previous receiver name opts for BinaryInstallationOptions (golint)
    • Line 270: warning: exported method GetOptions.Version should have comment or be unexported (golint)
    • Line 283: warning: exported type Release should have comment or be unexported (golint)
    • Line 287: warning: exported function ParseReleases should have comment or be unexported (golint)
    • Line 293: warning: exported type ChecksumFileEntry should have comment or be unexported (golint)
    • Line 300: warning: exported method ChecksumFileEntry.Ext should have comment or be unexported (golint)
    • Line 301: warning: exported method ChecksumFileEntry.BinVersion should have comment or be unexported (golint)
    • Line 302: warning: exported method ChecksumFileEntry.ProtVersion should have comment or be unexported (golint)
    • Line 303: warning: exported method ChecksumFileEntry.Os should have comment or be unexported (golint)
    • Line 304: warning: exported method ChecksumFileEntry.Arch should have comment or be unexported (golint)
    • Line 341: warning: exported function ParseChecksumFileEntries should have comment or be unexported (golint)
    • Line 346: warning: exported method Requirement.InstallLatest should have comment or be unexported (golint)
    • packer/fix/fixer_azure_exclude_from_latest.go
    • Line 13: warning: exported method FixerAzureExcludeFromLatest.DeprecatedOptions should have comment or be unexported (golint)
    • Line 19: warning: exported method FixerAzureExcludeFromLatest.Fix should have comment or be unexported (golint)
    • Line 70: warning: exported method FixerAzureExcludeFromLatest.Synopsis should have comment or be unexported (golint)
    • packer/post-processor/manifest/post-processor.go
    • Line 23: warning: exported type Config should have comment or be unexported (golint)
    • Line 41: warning: exported type PostProcessor should have comment or be unexported (golint)
    • Line 45: warning: exported type ManifestFile should have comment or be unexported (golint)
    • Line 50: warning: exported method PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 52: warning: exported method PostProcessor.Configure should have comment or be unexported (golint)
    • Line 76: warning: exported method PostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/provisioner/inspec/provisioner.go
    • Line 39: warning: exported var SupportedBackends should have comment or be unexported (golint)
    • Line 41: warning: exported type Config should have comment or be unexported (golint)
    • Line 66: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 74: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 76: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 199: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • Line 257: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • packer/packer/core.go
    • Line 52: warning: comment on exported type BuilderFunc should be of the form "BuilderFunc ..." (with optional leading article) (golint)
    • Line 55: warning: comment on exported type HookFunc should be of the form "HookFunc ..." (with optional leading article) (golint)
    • Line 58: warning: comment on exported type PostProcessorFunc should be of the form "PostProcessorFunc ..." (with optional leading article) (golint)
    • Line 61: warning: comment on exported type ProvisionerFunc should be of the form "ProvisionerFunc ..." (with optional leading article) (golint)
    • Line 64: warning: exported type BasicStore should have comment or be unexported (golint)
    • Line 69: warning: exported type BuilderStore should have comment or be unexported (golint)
    • Line 74: warning: exported type BuilderSet should have comment or be unexported (golint)
    • Line 79: warning: exported type ProvisionerStore should have comment or be unexported (golint)
    • Line 84: warning: exported type ProvisionerSet should have comment or be unexported (golint)
    • Line 89: warning: exported type PostProcessorStore should have comment or be unexported (golint)
    • Line 94: warning: exported type PostProcessorSet should have comment or be unexported (golint)
    • Line 99: warning: exported type DatasourceStore should have comment or be unexported (golint)
    • Line 104: warning: exported type DatasourceSet should have comment or be unexported (golint)
    • Line 130: warning: exported method Core.Initialize should have comment or be unexported (golint)
    • Line 158: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 183: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 243: warning: comment on exported method Core.GetBuilds should be of the form "GetBuilds ..." (golint)
    • Line 245: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 293: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 411: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 418: warning: exported var ConsoleHelp should have comment or be unexported (golint)
    • Line 435: warning: exported method Core.EvaluateExpression should have comment or be unexported (golint)
    • Line 435: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 464: warning: exported method Core.InspectConfig should have comment or be unexported (golint)
    • Line 464: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 587: warning: exported method Core.FixConfig should have comment or be unexported (golint)
    • Line 587: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 651: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 715: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • Line 849: warning: receiver name c should be consistent with previous receiver name core for Core (golint)
    • packer/command/cli.go
    • Line 16: warning: exported const ConfigTypeJSON should have comment (or a comment on this block) or be unexported (golint)
    • Line 28: warning: comment on exported method MetaArgs.GetConfigType should be of the form "GetConfigType ..." (golint)
    • Line 52: warning: comment on exported method MetaArgs.AddFlagSets should be of the form "AddFlagSets ..." (golint)
    • Line 73: warning: exported method BuildArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 96: warning: exported method InitArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 113: warning: exported method FixArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 125: warning: exported method ValidateArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 137: warning: exported method InspectArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 146: warning: exported method HCL2UpgradeArgs.AddFlagSets should have comment or be unexported (golint)
    • Line 160: warning: exported method FormatArgs.AddFlagSets should have comment or be unexported (golint)
    • packer/command/fix.go
    • Line 18: warning: exported type FixCommand should have comment or be unexported (golint)
    • Line 22: warning: exported method FixCommand.Run should have comment or be unexported (golint)
    • Line 34: warning: exported method FixCommand.ParseArgs should have comment or be unexported (golint)
    • Line 52: warning: exported method FixCommand.RunContext should have comment or be unexported (golint)
    • Line 132: warning: exported method FixCommand.Help should have comment or be unexported (golint)
    • Line 160: warning: exported method FixCommand.Synopsis should have comment or be unexported (golint)
    • Line 164: warning: exported method FixCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 168: warning: exported method FixCommand.AutocompleteFlags should have comment or be unexported (golint)
    • packer/provisioner/breakpoint/provisioner.go
    • Line 18: warning: exported type Config should have comment or be unexported (golint)
    • Line 27: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 31: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 33: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 49: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/post-processor/artifice/artifact.go
    • Line 10: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 12: warning: exported type Artifact should have comment or be unexported (golint)
    • Line 16: warning: exported function NewArtifact should have comment or be unexported (golint)
    • Line 33: warning: exported method Artifact.BuilderId should have comment or be unexported (golint)
    • Line 37: warning: exported method Artifact.Files should have comment or be unexported (golint)
    • Line 41: warning: exported method Artifact.Id should have comment or be unexported (golint)
    • Line 50: warning: exported method Artifact.State should have comment or be unexported (golint)
    • Line 54: warning: exported method Artifact.Destroy should have comment or be unexported (golint)
    • packer/fix/fixer_virtualbox_gaattach.go
    • Line 11: warning: exported method FixerVirtualBoxGAAttach.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerVirtualBoxGAAttach.Fix should have comment or be unexported (golint)
    • Line 67: warning: exported method FixerVirtualBoxGAAttach.Synopsis should have comment or be unexported (golint)
    • packer/builder/profitbricks/builder.go
    • Line 14: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 16: warning: exported type Builder should have comment or be unexported (golint)
    • Line 21: warning: exported method Builder.ConfigSpec should have comment or be unexported (golint)
    • Line 23: warning: exported method Builder.Prepare should have comment or be unexported (golint)
    • Line 32: warning: exported method Builder.Run should have comment or be unexported (golint)
    • packer/hcl2template/types.packer_config.go
    • Line 66: warning: exported type ValidationOptions should have comment or be unexported (golint)
    • Line 80: warning: exported type BlockContext should have comment or be unexported (golint)
    • Line 83: warning: exported const InputVariableContext should have comment (or a comment on this block) or be unexported (golint)
    • Line 141: warning: receiver name c should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 174: warning: receiver name c should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 219: warning: receiver name c should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 259: warning: receiver name c should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 556: warning: exported var PackerConsoleHelp should have comment or be unexported (golint)
    • Line 575: warning: exported method PackerConfig.EvaluateExpression should have comment or be unexported (golint)
    • Line 575: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 590: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 611: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 659: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 677: warning: exported method PackerConfig.FixConfig should have comment or be unexported (golint)
    • Line 677: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • Line 682: warning: exported method PackerConfig.InspectConfig should have comment or be unexported (golint)
    • Line 682: warning: receiver name p should be consistent with previous receiver name cfg for PackerConfig (golint)
    • packer/post-processor/manifest/artifact.go
    • Line 5: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 7: warning: exported type ArtifactFile should have comment or be unexported (golint)
    • Line 12: warning: exported type Artifact should have comment or be unexported (golint)
    • Line 22: warning: exported method Artifact.BuilderId should have comment or be unexported (golint)
    • Line 26: warning: exported method Artifact.Files should have comment or be unexported (golint)
    • Line 34: warning: exported method Artifact.Id should have comment or be unexported (golint)
    • Line 42: warning: exported method Artifact.State should have comment or be unexported (golint)
    • Line 46: warning: exported method Artifact.Destroy should have comment or be unexported (golint)
    • packer/fix/fixer_powershell_escapes.go
    • Line 13: warning: exported method FixerPowerShellEscapes.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerPowerShellEscapes.Fix should have comment or be unexported (golint)
    • Line 53: warning: don't use underscores in Go names; var env_vars should be envVars (golint)
    • Line 57: warning: don't use underscores in Go names; var env_vars_unescaped should be envVarsUnescaped (golint)
    • Line 58: warning: don't use underscores in Go names; range var env_var should be envVar (golint)
    • Line 76: warning: exported method FixerPowerShellEscapes.Synopsis should have comment or be unexported (golint)
    • packer/post-processor/checksum/post-processor.go
    • Line 24: warning: exported type Config should have comment or be unexported (golint)
    • Line 32: warning: exported type PostProcessor should have comment or be unexported (golint)
    • Line 55: warning: exported method PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 57: warning: exported method PostProcessor.Configure should have comment or be unexported (golint)
    • Line 98: warning: exported method PostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/hcl2template/types.datasource.go
    • Line 23: warning: exported type DatasourceRef should have comment or be unexported (golint)
    • Line 28: warning: exported type Datasources should have comment or be unexported (golint)
    • Line 30: warning: exported method DatasourceBlock.Ref should have comment or be unexported (golint)
    • Line 37: warning: exported method Datasources.Values should have comment or be unexported (golint)
    • packer/provisioner/windows-shell/provisioner.go
    • Line 3: warning: package comment should be of the form "Package shell ..." (golint)
    • Line 28: warning: comment on exported const DefaultRemotePath should be of the form "DefaultRemotePath ..." (golint)
    • Line 31: warning: exported type Config should have comment or be unexported (golint)
    • Line 44: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 49: warning: exported type ExecuteCommandTemplate should have comment or be unexported (golint)
    • Line 54: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 56: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 162: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/provisioner/shell/provisioner.go
    • Line 3: warning: package comment should be of the form "Package shell ..." (golint)
    • Line 30: warning: exported type Config should have comment or be unexported (golint)
    • Line 69: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 74: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 76: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 180: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/fix/fixer_virtualbox_rename.go
    • Line 10: warning: exported method FixerVirtualBoxRename.DeprecatedOptions should have comment or be unexported (golint)
    • Line 14: warning: exported method FixerVirtualBoxRename.Fix should have comment or be unexported (golint)
    • Line 80: warning: exported method FixerVirtualBoxRename.Synopsis should have comment or be unexported (golint)
    • packer/builder/profitbricks/step_create_server.go
    • Line 183: warning: receiver name d should be consistent with previous receiver name s for stepCreateServer (golint)
    • Line 203: warning: receiver name d should be consistent with previous receiver name s for stepCreateServer (golint)
    • Line 208: warning: receiver name d should be consistent with previous receiver name s for stepCreateServer (golint)
    • Line 215: warning: exported type RestError should have comment or be unexported (golint)
    • Line 220: warning: exported type Message should have comment or be unexported (golint)
    • Line 225: warning: receiver name d should be consistent with previous receiver name s for stepCreateServer (golint)
    • Line 246: warning: receiver name d should be consistent with previous receiver name s for stepCreateServer (golint)
    • packer/packer/plugin_client.go
    • Line 24: warning: comment on exported var Killed should be of the form "Killed ..." (golint)
    • Line 32: warning: comment on exported type PluginClient should be of the form "PluginClient ..." (with optional leading article) (golint)
    • Line 71: warning: comment on exported function CleanupClients should be of the form "CleanupClients ..." (golint)
    • Line 96: warning: comment on exported function NewClient should be of the form "NewClient ..." (golint)
    • Line 125: warning: comment on exported method PluginClient.Exited should be of the form "Exited ..." (golint)
    • Line 132: warning: comment on exported method PluginClient.Builder should be of the form "Builder ..." (golint)
    • Line 143: warning: comment on exported method PluginClient.Hook should be of the form "Hook ..." (golint)
    • Line 154: warning: comment on exported method PluginClient.PostProcessor should be of the form "PostProcessor ..." (golint)
    • Line 165: warning: comment on exported method PluginClient.Provisioner should be of the form "Provisioner ..." (golint)
    • Line 176: warning: comment on exported method PluginClient.Datasource should be of the form "Datasource ..." (golint)
    • Line 186: warning: comment on exported method PluginClient.Kill should be of the form "Kill ..." (golint)
    • Line 205: warning: comment on exported method PluginClient.Start should be of the form "Start ..." (golint)
    • Line 227: warning: don't use underscores in Go names; var stdout_r should be stdoutR (golint)
    • Line 227: warning: don't use underscores in Go names; var stdout_w should be stdoutW (golint)
    • Line 228: warning: don't use underscores in Go names; var stderr_r should be stderrR (golint)
    • Line 228: warning: don't use underscores in Go names; var stderr_w should be stderrW (golint)
    • Line 394: warning: exported method PluginClient.Client should have comment or be unexported (golint)
    • packer/provisioner/windows-restart/provisioner.go
    • Line 25: warning: exported var DefaultRestartCommand should have comment or be unexported (golint)
    • Line 26: warning: exported var DefaultRestartCheckCommand should have comment or be unexported (golint)
    • Line 28: warning: exported var TryCheckReboot should have comment or be unexported (golint)
    • Line 29: warning: exported var AbortReboot should have comment or be unexported (golint)
    • Line 31: warning: exported var DefaultRegistryKeys should have comment or be unexported (golint)
    • Line 37: warning: exported type Config should have comment or be unexported (golint)
    • Line 59: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 67: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 69: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 103: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/fix/fixer_iso_checksum_type_and_url.go
    • Line 11: warning: exported method FixerISOChecksumTypeAndURL.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerISOChecksumTypeAndURL.Fix should have comment or be unexported (golint)
    • Line 61: warning: exported method FixerISOChecksumTypeAndURL.Synopsis should have comment or be unexported (golint)
    • packer/builder/null/builder.go
    • Line 13: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 15: warning: exported type Builder should have comment or be unexported (golint)
    • Line 20: warning: exported method Builder.ConfigSpec should have comment or be unexported (golint)
    • Line 22: warning: exported method Builder.Prepare should have comment or be unexported (golint)
    • Line 31: warning: exported method Builder.Run should have comment or be unexported (golint)
    • packer/fix/fixer_vsphere_network_storage.go
    • Line 12: warning: exported method FixerVSphereNetworkDisk.DeprecatedOptions should have comment or be unexported (golint)
    • Line 19: warning: exported method FixerVSphereNetworkDisk.Fix should have comment or be unexported (golint)
    • Line 125: warning: exported method FixerVSphereNetworkDisk.Synopsis should have comment or be unexported (golint)
    • packer/builder/file/config.go
    • Line 14: warning: exported var ErrTargetRequired should have comment or be unexported (golint)
    • Line 15: warning: exported var ErrContentSourceConflict should have comment or be unexported (golint)
    • Line 17: warning: exported type Config should have comment or be unexported (golint)
    • Line 25: warning: exported method Config.Prepare should have comment or be unexported (golint)
    • packer/fix/fixer_amazon_private_ip.go
    • Line 15: warning: exported method FixerAmazonPrivateIP.DeprecatedOptions should have comment or be unexported (golint)
    • Line 21: warning: exported method FixerAmazonPrivateIP.Fix should have comment or be unexported (golint)
    • Line 78: warning: exported method FixerAmazonPrivateIP.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_pp_manifest_filename.go
    • Line 10: warning: exported method FixerManifestFilename.DeprecatedOptions should have comment or be unexported (golint)
    • Line 16: warning: exported method FixerManifestFilename.Fix should have comment or be unexported (golint)
    • Line 63: warning: exported method FixerManifestFilename.Synopsis should have comment or be unexported (golint)
    • packer/command/plugin.go
    • Line 36: warning: exported type PluginCommand should have comment or be unexported (golint)
    • Line 40: warning: exported var Builders should have comment or be unexported (golint)
    • Line 46: warning: exported var Provisioners should have comment or be unexported (golint)
    • Line 59: warning: exported var PostProcessors should have comment or be unexported (golint)
    • Line 67: warning: exported var Datasources should have comment or be unexported (golint)
    • Line 71: warning: exported method PluginCommand.Run should have comment or be unexported (golint)
    • Line 132: warning: exported method PluginCommand.Help should have comment or be unexported (golint)
    • Line 144: warning: exported method PluginCommand.Synopsis should have comment or be unexported (golint)
    • packer/fix/fixer_galaxy_command.go
    • Line 11: warning: exported method FixerGalaxyCommand.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerGalaxyCommand.Fix should have comment or be unexported (golint)
    • Line 74: warning: exported method FixerGalaxyCommand.Synopsis should have comment or be unexported (golint)
    • packer/packer/cmd_provisioner.go
    • Line 25: warning: receiver name c should be consistent with previous receiver name p for cmdProvisioner (golint)
    • Line 34: warning: receiver name c should be consistent with previous receiver name p for cmdProvisioner (golint)
    • Line 43: warning: receiver name c should be consistent with previous receiver name p for cmdProvisioner (golint)
    • packer/packer/post_processor_mock.go
    • Line 1: warning: package comment should be of the form "Package packer ..." (golint)
    • Line 28: warning: exported method MockPostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 30: warning: exported method MockPostProcessor.Configure should have comment or be unexported (golint)
    • Line 36: warning: exported method MockPostProcessor.PostProcess should have comment or be unexported (golint)
    • packer/hcl2template/types.hcl_provisioner.go
    • Line 25: warning: exported method HCL2Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 29: warning: exported method HCL2Provisioner.HCL2Prepare should have comment or be unexported (golint)
    • Line 66: warning: exported method HCL2Provisioner.Prepare should have comment or be unexported (golint)
    • Line 70: warning: exported method HCL2Provisioner.Provision should have comment or be unexported (golint)
    • packer/packer/build.go
    • Line 61: warning: comment on exported method CoreBuild.Name should be of the form "Name ..." (golint)
    • Line 158: warning: comment on exported method CoreBuild.Run should be of the form "Run ..." (golint)
    • Line 345: warning: exported method CoreBuild.SetDebug should have comment or be unexported (golint)
    • Line 353: warning: exported method CoreBuild.SetForce should have comment or be unexported (golint)
    • Line 361: warning: exported method CoreBuild.SetOnError should have comment or be unexported (golint)
    • packer/post-processor/compress/post-processor.go
    • Line 34: warning: exported var ErrWrongInputCount should have comment or be unexported (golint)
    • Line 40: warning: exported type Config should have comment or be unexported (golint)
    • Line 55: warning: exported type PostProcessor should have comment or be unexported (golint)
    • Line 59: warning: exported method PostProcessor.ConfigSpec should have comment or be unexported (golint)
    • Line 61: warning: exported method PostProcessor.Configure should have comment or be unexported (golint)
    • Line 109: warning: exported method PostProcessor.PostProcess should have comment or be unexported (golint)
    • Line 134: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • packer/packer/plugin-getter/checksum.go
    • Line 33: warning: exported type Checksum should have comment or be unexported (golint)
    • Line 37: warning: exported type FileChecksum should have comment or be unexported (golint)
    • Line 43: warning: exported type Checksummer should have comment or be unexported (golint)
    • Line 50: warning: exported method Checksummer.FileExt should have comment or be unexported (golint)
    • Line 93: warning: exported method Checksummer.Sum should have comment or be unexported (golint)
    • Line 101: warning: exported method Checksummer.Checksum should have comment or be unexported (golint)
    • packer/provisioner/sleep/provisioner.go
    • Line 14: warning: exported type Provisioner should have comment or be unexported (golint)
    • Line 20: warning: exported method Provisioner.ConfigSpec should have comment or be unexported (golint)
    • Line 22: warning: exported method Provisioner.FlatConfig should have comment or be unexported (golint)
    • Line 24: warning: exported method Provisioner.Prepare should have comment or be unexported (golint)
    • Line 28: warning: exported method Provisioner.Provision should have comment or be unexported (golint)
    • packer/command/version.go
    • Line 29: warning: exported method VersionCommand.Help should have comment or be unexported (golint)
    • Line 33: warning: exported method VersionCommand.Run should have comment or be unexported (golint)
    • Line 61: warning: exported method VersionCommand.Synopsis should have comment or be unexported (golint)
    • packer/post-processor/checksum/artifact.go
    • Line 9: warning: exported const BuilderId should have comment or be unexported (golint)
    • Line 11: warning: exported type Artifact should have comment or be unexported (golint)
    • Line 15: warning: exported function NewArtifact should have comment or be unexported (golint)
    • Line 19: warning: exported method Artifact.BuilderId should have comment or be unexported (golint)
    • Line 23: warning: exported method Artifact.Files should have comment or be unexported (golint)
    • Line 27: warning: exported method Artifact.Id should have comment or be unexported (golint)
    • Line 36: warning: exported method Artifact.State should have comment or be unexported (golint)
    • Line 40: warning: exported method Artifact.Destroy should have comment or be unexported (golint)
    • packer/command/hcl2_upgrade.go
    • Line 28: warning: exported type HCL2UpgradeCommand should have comment or be unexported (golint)
    • Line 32: warning: exported method HCL2UpgradeCommand.Run should have comment or be unexported (golint)
    • Line 44: warning: exported method HCL2UpgradeCommand.ParseArgs should have comment or be unexported (golint)
    • Line 128: warning: exported type BlockParser should have comment or be unexported (golint)
    • Line 133: warning: exported method HCL2UpgradeCommand.RunContext should have comment or be unexported (golint)
    • Line 262: warning: exported type UnhandleableArgumentError should have comment or be unexported (golint)
    • Line 645: warning: exported method HCL2UpgradeCommand.Help should have comment or be unexported (golint)
    • Line 664: warning: exported method HCL2UpgradeCommand.Synopsis should have comment or be unexported (golint)
    • Line 668: warning: exported method HCL2UpgradeCommand.AutocompleteArgs should have comment or be unexported (golint)
    • Line 672: warning: exported method HCL2UpgradeCommand.AutocompleteFlags should have comment or be unexported (golint)
    • Line 678: warning: exported type PackerParser should have comment or be unexported (golint)
    • Line 683: warning: exported method PackerParser.Parse should have comment or be unexported (golint)
    • Line 703: warning: exported type VariableParser should have comment or be unexported (golint)
    • Line 745: warning: exported method VariableParser.Parse should have comment or be unexported (golint)
    • Line 811: warning: exported type LocalsParser should have comment or be unexported (golint)
    • Line 816: warning: exported method LocalsParser.Parse should have comment or be unexported (golint)
    • Line 840: warning: exported type AmazonSecretsDatasourceParser should have comment or be unexported (golint)
    • Line 845: warning: exported method AmazonSecretsDatasourceParser.Parse should have comment or be unexported (golint)
    • Line 877: warning: exported type AmazonAmiDatasourceParser should have comment or be unexported (golint)
    • Line 883: warning: exported method AmazonAmiDatasourceParser.Parse should have comment or be unexported (golint)
    • Line 966: warning: exported type SourceParser should have comment or be unexported (golint)
    • Line 973: warning: exported method SourceParser.Parse should have comment or be unexported (golint)
    • Line 1008: warning: exported type BuildParser should have comment or be unexported (golint)
    • Line 1017: warning: exported method BuildParser.Parse should have comment or be unexported (golint)
    • Line 1069: warning: exported type ProvisionerParser should have comment or be unexported (golint)
    • Line 1074: warning: exported method ProvisionerParser.Parse should have comment or be unexported (golint)
    • Line 1126: warning: exported type PostProcessorParser should have comment or be unexported (golint)
    • Line 1131: warning: exported method PostProcessorParser.Parse should have comment or be unexported (golint)
    • Line 1179: warning: exported var PASSTHROUGHS should have comment or be unexported (golint)
    • packer/hcl2template/shim/mock.go
    • Line 11: warning: exported type NestedMockConfig should have comment or be unexported (golint)
    • Line 27: warning: exported type MockTag should have comment or be unexported (golint)
    • Line 32: warning: exported type MockConfig should have comment or be unexported (golint)
    • Line 39: warning: exported type NamedMapStringString should have comment or be unexported (golint)
    • Line 40: warning: exported type NamedString should have comment or be unexported (golint)
    • packer/fix/fixer_scaleway_access_key.go
    • Line 11: warning: exported method FixerScalewayAccessKey.DeprecatedOptions should have comment or be unexported (golint)
    • Line 17: warning: exported method FixerScalewayAccessKey.Fix should have comment or be unexported (golint)
    • Line 58: warning: exported method FixerScalewayAccessKey.Synopsis should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign99%

IneffAssign detects ineffectual assignments in Go code.


misspell97%

Misspell Finds commonly misspelled English words