Preparing report...

Report for github.com/hashicorp/vagrant-vmware-desktop

A    Great!    Found 58 issues across 83 files

Tweet

gofmt96%

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!


gocyclo92%

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.


golint34%

Golint is a linter for Go source code.

    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/vmnet_cli.go
    • Line 16: warning: exported type VmnetCli should have comment or be unexported (golint)
    • Line 24: warning: exported type VmnetCliExe should have comment or be unexported (golint)
    • Line 30: warning: exported function NewVmnetCli should have comment or be unexported (golint)
    • Line 41: warning: exported method VmnetCliExe.Start should have comment or be unexported (golint)
    • Line 52: warning: exported method VmnetCliExe.Stop should have comment or be unexported (golint)
    • Line 59: warning: exported method VmnetCliExe.Status should have comment or be unexported (golint)
    • Line 69: warning: exported method VmnetCliExe.Restart should have comment or be unexported (golint)
    • Line 80: warning: exported method VmnetCliExe.Configure should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/dhcp_lease_file.go
    • Line 14: warning: comment on exported const VMWARE_TIME_FORMAT should be of the form "VMWARE_TIME_FORMAT ..." (golint)
    • Line 16: warning: exported const VMWARE_LEASE_PATTERN should have comment or be unexported (golint)
    • Line 17: warning: exported const VMWARE_START_PATTERN should have comment or be unexported (golint)
    • Line 18: warning: exported const VMWARE_END_PATTERN should have comment or be unexported (golint)
    • Line 19: warning: exported const VMWARE_MAC_PATTERN should have comment or be unexported (golint)
    • Line 20: warning: exported const VMWARE_HOSTNAME_PATTERN should have comment or be unexported (golint)
    • Line 22: warning: comment on exported const MACOS_TIME_FORMAT should be of the form "MACOS_TIME_FORMAT ..." (golint)
    • Line 24: warning: exported const MACOS_LEASE_PATTERN should have comment or be unexported (golint)
    • Line 25: warning: exported const MACOS_START_PATTERN should have comment or be unexported (golint)
    • Line 26: warning: exported const MACOS_END_PATTERN should have comment or be unexported (golint)
    • Line 27: warning: exported const MACOS_MAC_PATTERN should have comment or be unexported (golint)
    • Line 28: warning: exported const MACOS_HOSTNAME_PATTERN should have comment or be unexported (golint)
    • Line 30: warning: comment on exported const VMWARE_LEASE_FILE_PREFIX should be of the form "VMWARE_LEASE_FILE_PREFIX ..." (golint)
    • Line 33: warning: exported type DhcpLeaseFile should have comment or be unexported (golint)
    • Line 46: warning: exported type DhcpEntry should have comment or be unexported (golint)
    • Line 54: warning: exported method DhcpEntry.NormalizeMac should have comment or be unexported (golint)
    • Line 69: warning: exported function LoadDhcpLeaseFile should have comment or be unexported (golint)
    • Line 105: warning: exported method DhcpLeaseFile.Load should have comment or be unexported (golint)
    • Line 162: warning: exported method DhcpLeaseFile.IpForMac should have comment or be unexported (golint)
    • Line 171: warning: exported method DhcpLeaseFile.AddEntry should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/routing_table.go
    • Line 9: warning: exported type NetworkInterface should have comment or be unexported (golint)
    • Line 14: warning: exported method NetworkInterface.Addrs should have comment or be unexported (golint)
    • Line 15: warning: exported method NetworkInterface.Name should have comment or be unexported (golint)
    • Line 17: warning: exported type InterfacesGetter should have comment or be unexported (golint)
    • Line 19: warning: exported type RoutingTable should have comment or be unexported (golint)
    • Line 26: warning: exported type RoutingDevice should have comment or be unexported (golint)
    • Line 32: warning: exported method RoutingDevice.Match should have comment or be unexported (golint)
    • Line 46: warning: exported function LoadRoutingTable should have comment or be unexported (golint)
    • Line 69: warning: exported method RoutingTable.Load should have comment or be unexported (golint)
    • Line 90: warning: exported method RoutingTable.DeviceByName should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/command/core.go
    • Line 18: warning: exported type ForceExit should have comment or be unexported (golint)
    • Line 22: warning: exported const ENV_VAR_PREFIX should have comment or be unexported (golint)
    • Line 24: warning: exported function Commands should have comment or be unexported (golint)
    • Line 35: warning: comment on exported type Command should be of the form "Command ..." (with optional leading article) (golint)
    • Line 47: warning: exported type Config should have comment or be unexported (golint)
    • Line 61: warning: exported type ConfigFile should have comment or be unexported (golint)
    • Line 67: warning: exported method Command.Help should have comment or be unexported (golint)
    • Line 76: warning: exported method Command.Synopsis should have comment or be unexported (golint)
    • Line 80: warning: exported method Command.Run should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/util/shutdown.go
    • Line 7: warning: exported var L should have comment or be unexported (golint)
    • Line 8: warning: exported var Shutdown should have comment or be unexported (golint)
    • Line 9: warning: exported var ShutdownTasks should have comment or be unexported (golint)
    • Line 11: warning: exported function RegisterShutdownTask should have comment or be unexported (golint)
    • Line 17: warning: exported function RunShutdownTasks should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/core.go
    • Line 14: warning: exported type VmwareInfo should have comment or be unexported (golint)
    • Line 22: warning: exported method VmwareInfo.IsStandard should have comment or be unexported (golint)
    • Line 26: warning: exported method VmwareInfo.IsProfessional should have comment or be unexported (golint)
    • Line 53: warning: exported method VmwareInfo.Normalize should have comment or be unexported (golint)
    • Line 61: warning: exported type Vmnet should have comment or be unexported (golint)
    • Line 69: warning: exported type Vmnets should have comment or be unexported (golint)
    • Line 74: warning: exported type PortFwdGuest should have comment or be unexported (golint)
    • Line 79: warning: exported type PortFwd should have comment or be unexported (golint)
    • Line 87: warning: exported method PortFwd.Matches should have comment or be unexported (golint)
    • Line 94: warning: exported type PortFwds should have comment or be unexported (golint)
    • Line 99: warning: exported type MacToIp should have comment or be unexported (golint)
    • Line 105: warning: exported type MacToIps should have comment or be unexported (golint)
    • Line 110: warning: exported const FUSION_ADVANCED_MAJOR_MIN should have comment or be unexported (golint)
    • Line 112: warning: exported type Driver should have comment or be unexported (golint)
    • Line 138: warning: exported function CreateDriver should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/vmrest.go
    • Line 35: warning: exported var Shutdown should have comment or be unexported (golint)
    • Line 37: warning: exported type Client should have comment or be unexported (golint)
    • Line 41: warning: exported type VmrestDriver should have comment or be unexported (golint)
    • Line 55: warning: don't use underscores in Go names; struct field config_path should be configPath (golint)
    • Line 70: warning: exported const HOME_DIR_ENV should have comment or be unexported (golint)
    • Line 71: warning: exported const VMREST_VERSION_CONSTRAINT should have comment or be unexported (golint)
    • Line 72: warning: exported const VMREST_URL should have comment or be unexported (golint)
    • Line 73: warning: exported const VMREST_CONFIG should have comment or be unexported (golint)
    • Line 74: warning: exported const WINDOWS_VMREST_CONFIG should have comment or be unexported (golint)
    • Line 75: warning: exported const VMREST_CONTENT_TYPE should have comment or be unexported (golint)
    • Line 76: warning: exported const VMREST_VAGRANT_DESC should have comment or be unexported (golint)
    • Line 77: warning: exported const VMREST_KEEPALIVE_SECONDS should have comment or be unexported (golint)
    • Line 79: warning: exported const VMWARE_NETDEV_PREFIX should have comment or be unexported (golint)
    • Line 80: warning: exported const VAGRANT_NETDEV_PREFIX should have comment or be unexported (golint)
    • Line 81: warning: exported const APIPA_CIDR should have comment or be unexported (golint)
    • Line 409: warning: exported function NewVmrest should have comment or be unexported (golint)
    • Line 420: warning: exported function NewVmrestDriver should have comment or be unexported (golint)
    • Line 471: warning: exported method VmrestDriver.Vmnets should have comment or be unexported (golint)
    • Line 484: warning: exported method VmrestDriver.AddVmnet should have comment or be unexported (golint)
    • Line 516: warning: exported method VmrestDriver.UpdateVmnet should have comment or be unexported (golint)
    • Line 526: warning: exported method VmrestDriver.DeleteVmnet should have comment or be unexported (golint)
    • Line 538: warning: exported method VmrestDriver.PortFwds should have comment or be unexported (golint)
    • Line 617: warning: exported method VmrestDriver.AddPortFwd should have comment or be unexported (golint)
    • Line 672: warning: exported method VmrestDriver.DeletePortFwd should have comment or be unexported (golint)
    • Line 705: warning: exported method VmrestDriver.LookupDhcpAddress should have comment or be unexported (golint)
    • Line 709: warning: exported method VmrestDriver.ReserveDhcpAddress should have comment or be unexported (golint)
    • Line 731: warning: exported method VmrestDriver.LoadNetworkingFile should have comment or be unexported (golint)
    • Line 735: warning: exported method VmrestDriver.VerifyVmnet should have comment or be unexported (golint)
    • Line 739: warning: comment on exported method VmrestDriver.Do should be of the form "Do ..." (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/command/service_install.go
    • Line 12: warning: exported const RUNIT_DIR should have comment or be unexported (golint)
    • Line 14: warning: exported type ServiceInstallCommand should have comment or be unexported (golint)
    • Line 19: warning: exported type ServiceInstallConfig should have comment or be unexported (golint)
    • Line 37: warning: exported method ServiceInstallConfig.Prepare should have comment or be unexported (golint)
    • Line 45: warning: comment on exported type ServiceInstallConfigFile should be of the form "ServiceInstallConfigFile ..." (with optional leading article) (golint)
    • Line 51: warning: exported method ServiceInstallConfigFile.Prepare should have comment or be unexported (golint)
    • Line 57: warning: exported function BuildServiceInstallCommand should have comment or be unexported (golint)
    • Line 88: warning: exported method ServiceInstallCommand.Run should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/server/api.go
    • Line 20: warning: exported type Api should have comment or be unexported (golint)
    • Line 35: warning: exported function Create should have comment or be unexported (golint)
    • Line 87: warning: exported method Api.Start should have comment or be unexported (golint)
    • Line 110: warning: exported method Api.Stop should have comment or be unexported (golint)
    • Line 140: warning: exported method Api.RequestHandler should have comment or be unexported (golint)
    • Line 152: warning: exported method Api.Inflight should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/networking_file.go
    • Line 18: warning: exported const ADAPTER_PATTERN should have comment or be unexported (golint)
    • Line 19: warning: exported const DHCP_RES_PATTERN should have comment or be unexported (golint)
    • Line 20: warning: exported const PORTFWD_PATTERN should have comment or be unexported (golint)
    • Line 22: warning: exported type NetworkingFile should have comment or be unexported (golint)
    • Line 43: warning: exported type VMWareNetworkingFile should have comment or be unexported (golint)
    • Line 51: warning: exported type PortFwd should have comment or be unexported (golint)
    • Line 61: warning: exported type DhcpReservation should have comment or be unexported (golint)
    • Line 67: warning: exported type Device should have comment or be unexported (golint)
    • Line 77: warning: comment on exported function LoadNetworkingFile should be of the form "LoadNetworkingFile ..." (golint)
    • Line 99: warning: exported method VMWareNetworkingFile.GetPortFwds should have comment or be unexported (golint)
    • Line 103: warning: exported method VMWareNetworkingFile.GetPath should have comment or be unexported (golint)
    • Line 107: warning: exported method VMWareNetworkingFile.GetDevices should have comment or be unexported (golint)
    • Line 111: warning: comment on exported method VMWareNetworkingFile.Merge should be of the form "Merge ..." (golint)
    • Line 135: warning: exported method VMWareNetworkingFile.MergeFwds should have comment or be unexported (golint)
    • Line 152: warning: comment on exported method VMWareNetworkingFile.Save should be of the form "Save ..." (golint)
    • Line 242: warning: comment on exported method VMWareNetworkingFile.AddDhcpReservation should be of the form "AddDhcpReservation ..." (golint)
    • Line 263: warning: comment on exported method VMWareNetworkingFile.LookupDhcpReservation should be of the form "LookupDhcpReservation ..." (golint)
    • Line 271: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 274: warning: comment on exported method VMWareNetworkingFile.AddPortFwd should be of the form "AddPortFwd ..." (golint)
    • Line 295: warning: comment on exported method VMWareNetworkingFile.RemovePortFwd should be of the form "RemovePortFwd ..." (golint)
    • Line 314: warning: comment on exported method VMWareNetworkingFile.RemoveDeviceByName should be of the form "RemoveDeviceByName ..." (golint)
    • Line 325: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 332: warning: comment on exported method VMWareNetworkingFile.RemoveDeviceBySlot should be of the form "RemoveDeviceBySlot ..." (golint)
    • Line 343: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 350: warning: comment on exported method VMWareNetworkingFile.GetDeviceByName should be of the form "GetDeviceByName ..." (golint)
    • Line 361: warning: comment on exported method VMWareNetworkingFile.GetDeviceBySlot should be of the form "GetDeviceBySlot ..." (golint)
    • Line 372: warning: comment on exported method VMWareNetworkingFile.CreateDevice should be of the form "CreateDevice ..." (golint)
    • Line 407: warning: comment on exported method VMWareNetworkingFile.Load should be of the form "Load ..." (golint)
    • Line 555: warning: comment on exported method VMWareNetworkingFile.PortfwdExists should be of the form "PortfwdExists ..." (golint)
    • Line 565: warning: exported method VMWareNetworkingFile.HostPortFwd should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/simple.go
    • Line 17: warning: comment on exported type SimpleDriver should be of the form "SimpleDriver ..." (with optional leading article) (golint)
    • Line 29: warning: exported function NewSimpleDriver should have comment or be unexported (golint)
    • Line 42: warning: exported method SimpleDriver.AddVmnet should have comment or be unexported (golint)
    • Line 62: warning: exported method SimpleDriver.UpdateVmnet should have comment or be unexported (golint)
    • Line 69: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 82: warning: exported method SimpleDriver.DeleteVmnet should have comment or be unexported (golint)
    • Line 94: warning: comment on exported method SimpleDriver.LookupDhcpAddress should be of the form "LookupDhcpAddress ..." (golint)
    • Line 114: warning: exported method SimpleDriver.ReserveDhcpAddress should have comment or be unexported (golint)
    • Line 126: warning: exported method SimpleDriver.AddPortFwd should have comment or be unexported (golint)
    • Line 151: warning: exported method SimpleDriver.DeletePortFwd should have comment or be unexported (golint)
    • Line 187: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/nat_file_mock.go
    • Line 3: warning: exported type NatFileMock should have comment or be unexported (golint)
    • Line 8: warning: exported function LoadNatFileMock should have comment or be unexported (golint)
    • Line 22: warning: exported method NatFileMock.Load should have comment or be unexported (golint)
    • Line 26: warning: exported method NatFileMock.Save should have comment or be unexported (golint)
    • Line 30: warning: exported method NatFileMock.GetSection should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/settings/nat.go
    • Line 16: warning: exported type NatInfo should have comment or be unexported (golint)
    • Line 20: warning: exported type NAT should have comment or be unexported (golint)
    • Line 27: warning: exported function LoadNATSettings should have comment or be unexported (golint)
    • Line 38: warning: exported method NAT.PortFwds should have comment or be unexported (golint)
    • Line 42: warning: exported method NAT.Init should have comment or be unexported (golint)
    • Line 50: warning: exported method NAT.Clear should have comment or be unexported (golint)
    • Line 56: warning: exported method NAT.MultiAdd should have comment or be unexported (golint)
    • Line 65: warning: exported method NAT.Add should have comment or be unexported (golint)
    • Line 83: warning: exported method NAT.Remove should have comment or be unexported (golint)
    • Line 100: warning: exported method NAT.Reload should have comment or be unexported (golint)
    • Line 123: warning: exported method NAT.Save should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/command/rest_api.go
    • Line 16: warning: exported var Shutdown should have comment or be unexported (golint)
    • Line 18: warning: exported const DEFAULT_RESTAPI_PORT should have comment or be unexported (golint)
    • Line 20: warning: comment on exported type RestApiCommand should be of the form "RestApiCommand ..." (with optional leading article) (golint)
    • Line 26: warning: exported type RestApiConfig should have comment or be unexported (golint)
    • Line 39: warning: exported function BuildRestApiCommand should have comment or be unexported (golint)
    • Line 63: warning: exported method RestApiCommand.Run should have comment or be unexported (golint)
    • Line 120: warning: don't use underscores in Go names; var attempt_vmrest should be attemptVmrest (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/server/regexp_handler.go
    • Line 16: warning: exported const API_CONTENT_TYPE should have comment or be unexported (golint)
    • Line 23: warning: exported type RegexpHandler should have comment or be unexported (golint)
    • Line 31: warning: exported function NewRegexpHandler should have comment or be unexported (golint)
    • Line 38: warning: exported method RegexpHandler.HandleFunc should have comment or be unexported (golint)
    • Line 57: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 66: warning: exported type StandardResponse should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/internal/service/port_forwarding.go
    • Line 15: warning: exported type Forward should have comment or be unexported (golint)
    • Line 25: warning: exported method Forward.Deactivate should have comment or be unexported (golint)
    • Line 34: warning: exported method Forward.Activate should have comment or be unexported (golint)
    • Line 98: warning: exported type PortForwarding should have comment or be unexported (golint)
    • Line 107: warning: exported function NewPortForwarding should have comment or be unexported (golint)
    • Line 117: warning: comment on exported method PortForwarding.Load should be of the form "Load ..." (golint)
    • Line 140: warning: exported method PortForwarding.Start should have comment or be unexported (golint)
    • Line 159: warning: exported method PortForwarding.Stop should have comment or be unexported (golint)
    • Line 174: warning: exported method PortForwarding.Add should have comment or be unexported (golint)
    • Line 208: warning: exported method PortForwarding.Remove should have comment or be unexported (golint)
    • Line 232: warning: exported method PortForwarding.Fwds should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/settings/port_forwarding.go
    • Line 14: warning: exported type Address should have comment or be unexported (golint)
    • Line 20: warning: exported method Address.Network should have comment or be unexported (golint)
    • Line 28: warning: exported method Address.Equal should have comment or be unexported (golint)
    • Line 34: warning: exported type Forward should have comment or be unexported (golint)
    • Line 40: warning: exported method Forward.Equal should have comment or be unexported (golint)
    • Line 46: warning: exported type PortForwarding should have comment or be unexported (golint)
    • Line 54: warning: exported function LoadPortForwardingSettings should have comment or be unexported (golint)
    • Line 67: warning: exported method PortForwarding.Init should have comment or be unexported (golint)
    • Line 75: warning: exported method PortForwarding.Add should have comment or be unexported (golint)
    • Line 88: warning: exported method PortForwarding.Delete should have comment or be unexported (golint)
    • Line 100: warning: exported method PortForwarding.Save should have comment or be unexported (golint)
    • Line 144: warning: exported method PortForwarding.Reload should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/nat_file.go
    • Line 14: warning: exported type NatFile should have comment or be unexported (golint)
    • Line 20: warning: exported type VMWareNatFile should have comment or be unexported (golint)
    • Line 26: warning: exported type NatSection should have comment or be unexported (golint)
    • Line 31: warning: exported type NatEntry should have comment or be unexported (golint)
    • Line 36: warning: comment on exported function LoadNatFile should be of the form "LoadNatFile ..." (golint)
    • Line 56: warning: comment on exported method VMWareNatFile.Load should be of the form "Load ..." (golint)
    • Line 135: warning: comment on exported method VMWareNatFile.GetSection should be of the form "GetSection ..." (golint)
    • Line 145: warning: comment on exported method NatSection.DeleteEntry should be of the form "DeleteEntry ..." (golint)
    • Line 155: warning: comment on exported method NatSection.DeleteEntryAt should be of the form "DeleteEntryAt ..." (golint)
    • Line 164: warning: comment on exported method NatEntry.Match should be of the form "Match ..." (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/certificate.go
    • Line 18: warning: comment on exported const CERTIFICATE_EXPIRES_IN should be of the form "CERTIFICATE_EXPIRES_IN ..." (golint)
    • Line 21: warning: exported type CertificatePaths should have comment or be unexported (golint)
    • Line 28: warning: exported function GenerateCertificate should have comment or be unexported (golint)
    • Line 31: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 37: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 42: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 47: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 68: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 73: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 77: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 81: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 85: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 91: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 109: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 115: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 121: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 126: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 131: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 136: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 142: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 147: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 152: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 157: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 164: warning: comment on exported function GetCertificatePaths should be of the form "GetCertificatePaths ..." (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/utility/networking_file_mock.go
    • Line 9: warning: exported type NetworkingFileMock should have comment or be unexported (golint)
    • Line 16: warning: comment on exported function LoadNetworkingFileMock should be of the form "LoadNetworkingFileMock ..." (golint)
    • Line 28: warning: exported method NetworkingFileMock.GetPortFwds should have comment or be unexported (golint)
    • Line 32: warning: exported method NetworkingFileMock.GetPath should have comment or be unexported (golint)
    • Line 36: warning: exported method NetworkingFileMock.GetDevices should have comment or be unexported (golint)
    • Line 40: warning: exported method NetworkingFileMock.Merge should have comment or be unexported (golint)
    • Line 44: warning: exported method NetworkingFileMock.MergeFwds should have comment or be unexported (golint)
    • Line 48: warning: exported method NetworkingFileMock.Save should have comment or be unexported (golint)
    • Line 52: warning: exported method NetworkingFileMock.AddDhcpReservation should have comment or be unexported (golint)
    • Line 56: warning: exported method NetworkingFileMock.LookupDhcpReservation should have comment or be unexported (golint)
    • Line 63: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 66: warning: comment on exported method NetworkingFileMock.AddPortFwd should be of the form "AddPortFwd ..." (golint)
    • Line 71: warning: comment on exported method NetworkingFileMock.RemovePortFwd should be of the form "RemovePortFwd ..." (golint)
    • Line 76: warning: comment on exported method NetworkingFileMock.RemoveDeviceByName should be of the form "RemoveDeviceByName ..." (golint)
    • Line 81: warning: comment on exported method NetworkingFileMock.RemoveDeviceBySlot should be of the form "RemoveDeviceBySlot ..." (golint)
    • Line 86: warning: comment on exported method NetworkingFileMock.GetDeviceByName should be of the form "GetDeviceByName ..." (golint)
    • Line 97: warning: comment on exported method NetworkingFileMock.GetDeviceBySlot should be of the form "GetDeviceBySlot ..." (golint)
    • Line 108: warning: exported method NetworkingFileMock.CreateDevice should have comment or be unexported (golint)
    • Line 130: warning: exported method NetworkingFileMock.Load should have comment or be unexported (golint)
    • Line 134: warning: exported method NetworkingFileMock.PortfwdExists should have comment or be unexported (golint)
    • Line 143: warning: exported method NetworkingFileMock.HostPortFwd should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/vmrun_mock.go
    • Line 3: warning: exported type VmrunMock should have comment or be unexported (golint)
    • Line 7: warning: exported type VmrunResponse should have comment or be unexported (golint)
    • Line 12: warning: exported method VmrunMock.AddResponse should have comment or be unexported (golint)
    • Line 16: warning: exported method VmrunMock.GenerateResponse should have comment or be unexported (golint)
    • Line 25: warning: exported method VmrunMock.RunningVms should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/base.go
    • Line 21: warning: exported const PORTFWD_PREFIX should have comment or be unexported (golint)
    • Line 22: warning: exported const VMWARE_VERSION_PATTERN should have comment or be unexported (golint)
    • Line 24: warning: exported type BaseDriver should have comment or be unexported (golint)
    • Line 40: warning: exported function NewBaseDriver should have comment or be unexported (golint)
    • Line 101: warning: exported method BaseDriver.Settings should have comment or be unexported (golint)
    • Line 105: warning: exported method BaseDriver.Validated should have comment or be unexported (golint)
    • Line 109: warning: exported method BaseDriver.ValidationReason should have comment or be unexported (golint)
    • Line 113: warning: exported method BaseDriver.VmwarePaths should have comment or be unexported (golint)
    • Line 117: warning: exported method BaseDriver.Path should have comment or be unexported (golint)
    • Line 124: warning: exported method BaseDriver.LoadNatFile should have comment or be unexported (golint)
    • Line 128: warning: exported method BaseDriver.LoadNetworkingFile should have comment or be unexported (golint)
    • Line 146: warning: exported method BaseDriver.EnableInternalPortForwarding should have comment or be unexported (golint)
    • Line 170: warning: exported method BaseDriver.InternalPortForwarding should have comment or be unexported (golint)
    • Line 174: warning: exported method BaseDriver.InternalPortFwds should have comment or be unexported (golint)
    • Line 186: warning: exported method BaseDriver.AddInternalPortForward should have comment or be unexported (golint)
    • Line 193: warning: exported method BaseDriver.DeleteInternalPortForward should have comment or be unexported (golint)
    • Line 251: warning: comment on exported method BaseDriver.PrunePortFwds should be of the form "PrunePortFwds ..." (golint)
    • Line 282: warning: comment on exported method BaseDriver.VerifyVmnet should be of the form "VerifyVmnet ..." (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/advanced.go
    • Line 18: warning: exported type AdvancedDriver should have comment or be unexported (golint)
    • Line 23: warning: exported function NewAdvancedDriver should have comment or be unexported (golint)
    • Line 45: warning: exported method AdvancedDriver.AddVmnet should have comment or be unexported (golint)
    • Line 103: warning: exported method AdvancedDriver.UpdateVmnet should have comment or be unexported (golint)
    • Line 189: warning: exported method AdvancedDriver.DeleteVmnet should have comment or be unexported (golint)
    • Line 204: warning: comment on exported method AdvancedDriver.LookupDhcpAddress should be of the form "LookupDhcpAddress ..." (golint)
    • Line 218: warning: exported method AdvancedDriver.ReserveDhcpAddress should have comment or be unexported (golint)
    • Line 235: warning: comment on exported method AdvancedDriver.DeletePortFwd should be of the form "DeletePortFwd ..." (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/driver/driver_mock.go
    • Line 7: warning: exported type MockDriver should have comment or be unexported (golint)
    • Line 9: warning: exported method MockDriver.Path should have comment or be unexported (golint)
    • Line 13: warning: exported method MockDriver.Vmnets should have comment or be unexported (golint)
    • Line 17: warning: exported method MockDriver.AddVmnet should have comment or be unexported (golint)
    • Line 21: warning: exported method MockDriver.UpdateVmnet should have comment or be unexported (golint)
    • Line 25: warning: exported method MockDriver.DeleteVmnet should have comment or be unexported (golint)
    • Line 29: warning: exported method MockDriver.PortFwds should have comment or be unexported (golint)
    • Line 33: warning: exported method MockDriver.AddPortFwd should have comment or be unexported (golint)
    • Line 37: warning: exported method MockDriver.DeletePortFwd should have comment or be unexported (golint)
    • Line 41: warning: exported method MockDriver.PrunePortFwds should have comment or be unexported (golint)
    • Line 45: warning: exported method MockDriver.LookupDhcpAddress should have comment or be unexported (golint)
    • Line 49: warning: exported method MockDriver.ReserveDhcpAddress should have comment or be unexported (golint)
    • Line 53: warning: exported method MockDriver.VmwareInfo should have comment or be unexported (golint)
    • Line 57: warning: exported method MockDriver.VmwarePaths should have comment or be unexported (golint)
    • Line 62: warning: exported method MockDriver.LoadNetworkingFile should have comment or be unexported (golint)
    • Line 66: warning: exported method MockDriver.Validated should have comment or be unexported (golint)
    • Line 70: warning: exported method MockDriver.Validate should have comment or be unexported (golint)
    • Line 74: warning: exported method MockDriver.ValidationReason should have comment or be unexported (golint)
    • Line 78: warning: exported method MockDriver.VerifyVmnet should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/vmnet_cli_mock.go
    • Line 3: warning: exported type VmnetCliMock should have comment or be unexported (golint)
    • Line 13: warning: exported method VmnetCliMock.Start should have comment or be unexported (golint)
    • Line 21: warning: exported method VmnetCliMock.Stop should have comment or be unexported (golint)
    • Line 29: warning: exported method VmnetCliMock.Status should have comment or be unexported (golint)
    • Line 38: warning: exported method VmnetCliMock.Restart should have comment or be unexported (golint)
    • Line 46: warning: exported method VmnetCliMock.Configure should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/vnetlib_mock.go
    • Line 3: warning: exported type VnetlibMock should have comment or be unexported (golint)
    • Line 45: warning: exported type CreateDeviceResponse should have comment or be unexported (golint)
    • Line 50: warning: exported type SetSubnetAddressRequest should have comment or be unexported (golint)
    • Line 55: warning: exported type SetSubnetMaskRequest should have comment or be unexported (golint)
    • Line 60: warning: exported type SetNATRequest should have comment or be unexported (golint)
    • Line 65: warning: exported type SetDHCPRequest should have comment or be unexported (golint)
    • Line 70: warning: exported type LookupReservedAddressResponse should have comment or be unexported (golint)
    • Line 75: warning: exported type LookupReservedAddressRequest should have comment or be unexported (golint)
    • Line 80: warning: exported type ReserveAddressRequest should have comment or be unexported (golint)
    • Line 86: warning: exported type DeletePortFwdRequest should have comment or be unexported (golint)
    • Line 92: warning: exported type GetUnusedDeviceResponse should have comment or be unexported (golint)
    • Line 97: warning: exported method VnetlibMock.CreateDevice should have comment or be unexported (golint)
    • Line 109: warning: exported method VnetlibMock.DeleteDevice should have comment or be unexported (golint)
    • Line 119: warning: exported method VnetlibMock.SetSubnetAddress should have comment or be unexported (golint)
    • Line 129: warning: exported method VnetlibMock.SetSubnetMask should have comment or be unexported (golint)
    • Line 139: warning: exported method VnetlibMock.SetNAT should have comment or be unexported (golint)
    • Line 149: warning: exported method VnetlibMock.UpdateDeviceNAT should have comment or be unexported (golint)
    • Line 159: warning: exported method VnetlibMock.StatusNAT should have comment or be unexported (golint)
    • Line 170: warning: exported method VnetlibMock.StartNAT should have comment or be unexported (golint)
    • Line 180: warning: exported method VnetlibMock.StopNAT should have comment or be unexported (golint)
    • Line 190: warning: exported method VnetlibMock.SetDHCP should have comment or be unexported (golint)
    • Line 200: warning: exported method VnetlibMock.StatusDHCP should have comment or be unexported (golint)
    • Line 211: warning: exported method VnetlibMock.StartDHCP should have comment or be unexported (golint)
    • Line 221: warning: exported method VnetlibMock.StopDHCP should have comment or be unexported (golint)
    • Line 231: warning: exported method VnetlibMock.LookupReservedAddress should have comment or be unexported (golint)
    • Line 243: warning: exported method VnetlibMock.ReserveAddress should have comment or be unexported (golint)
    • Line 253: warning: exported method VnetlibMock.EnableDevice should have comment or be unexported (golint)
    • Line 262: warning: exported method VnetlibMock.DisableDevice should have comment or be unexported (golint)
    • Line 272: warning: exported method VnetlibMock.UpdateDevice should have comment or be unexported (golint)
    • Line 282: warning: exported method VnetlibMock.DeletePortFwd should have comment or be unexported (golint)
    • Line 292: warning: exported method VnetlibMock.GetUnusedDevice should have comment or be unexported (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/launchctl.go
    • Line 12: warning: exported const LAUNCHCTL_PATH should have comment or be unexported (golint)
    • Line 14: warning: exported type Launchctl should have comment or be unexported (golint)
    • Line 19: warning: exported type LaunchctlExe should have comment or be unexported (golint)
    • Line 23: warning: exported function NewLaunchctl should have comment or be unexported (golint)
    • Line 31: warning: exported method LaunchctlExe.Load should have comment or be unexported (golint)
    • Line 41: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 48: warning: exported method LaunchctlExe.Unload should have comment or be unexported (golint)
    • Line 58: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • vagrant-vmware-desktop/go_src/vagrant-vmware-utility/service/vnetlib.go
    • Line 12: warning: exported type Vnetlib should have comment or be unexported (golint)
    • Line 35: warning: exported type VnetlibExe should have comment or be unexported (golint)
    • Line 41: warning: exported function NewVnetlib should have comment or be unexported (golint)
    • Line 52: warning: comment on exported method VnetlibExe.CreateDevice should be of the form "CreateDevice ..." (golint)
    • Line 74: warning: exported method VnetlibExe.DeleteDevice should have comment or be unexported (golint)
    • Line 87: warning: exported method VnetlibExe.SetSubnetAddress should have comment or be unexported (golint)
    • Line 100: warning: exported method VnetlibExe.SetSubnetMask should have comment or be unexported (golint)
    • Line 113: warning: exported method VnetlibExe.SetNAT should have comment or be unexported (golint)
    • Line 132: warning: exported method VnetlibExe.SetDHCP should have comment or be unexported (golint)
    • Line 151: warning: exported method VnetlibExe.LookupReservedAddress should have comment or be unexported (golint)
    • Line 156: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 163: warning: exported method VnetlibExe.ReserveAddress should have comment or be unexported (golint)
    • Line 178: warning: exported method VnetlibExe.EnableDevice should have comment or be unexported (golint)
    • Line 191: warning: exported method VnetlibExe.DisableDevice should have comment or be unexported (golint)
    • Line 204: warning: exported method VnetlibExe.UpdateDevice should have comment or be unexported (golint)
    • Line 217: warning: exported method VnetlibExe.UpdateDeviceNAT should have comment or be unexported (golint)
    • Line 230: warning: exported method VnetlibExe.DeletePortFwd should have comment or be unexported (golint)
    • Line 243: warning: comment on exported method VnetlibExe.StatusNAT should be of the form "StatusNAT ..." (golint)
    • Line 251: warning: exported method VnetlibExe.StatusDHCP should have comment or be unexported (golint)
    • Line 258: warning: exported method VnetlibExe.StartNAT should have comment or be unexported (golint)
    • Line 271: warning: exported method VnetlibExe.StartDHCP should have comment or be unexported (golint)
    • Line 284: warning: exported method VnetlibExe.StopNAT should have comment or be unexported (golint)
    • Line 297: warning: exported method VnetlibExe.StopDHCP should have comment or be unexported (golint)
    • Line 310: warning: comment on exported method VnetlibExe.GetUnusedDevice should be of the form "GetUnusedDevice ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign93%

IneffAssign detects ineffectual assignments in Go code.


misspell98%

Misspell Finds commonly misspelled English words