gofmt100%

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

No problems detected. Good job!

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo98%

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.

• newrelic-cli/internal/install/recipe_installer_guided.go
  • Line 19: warning: cyclomatic complexity 17 of function (*RecipeInstaller).guidedInstall() is high (> 15) (gocyclo)
  • Line 265: warning: cyclomatic complexity 17 of function (*RecipeInstaller).filterIntegrations() is high (> 15) (gocyclo)

• newrelic-cli/cmd/newrelic/command.go
  • Line 43: warning: cyclomatic complexity 17 of function initializeProfile() is high (> 15) (gocyclo)

• newrelic-cli/internal/install/types/discovery_manifest.go
  • Line 39: warning: cyclomatic complexity 16 of function (*DiscoveryManifest).ConstrainRecipes() is high (> 15) (gocyclo)

golint71%

Golint is a linter for Go source code.

• newrelic-cli/internal/install/execution/install_status.go
  • Line 15: warning: comment on exported type InstallStatus should be of the form "InstallStatus ..." (with optional leading article) (golint)
  • Line 41: warning: exported type RecipeStatus should have comment or be unexported (golint)
  • Line 51: warning: exported type RecipeStatusType should have comment or be unexported (golint)
  • Line 53: warning: exported var RecipeStatusTypes should have comment or be unexported (golint)
  • Line 71: warning: exported type StatusError should have comment or be unexported (golint)
  • Line 77: warning: exported function NewInstallStatus should have comment or be unexported (golint)
  • Line 89: warning: exported method InstallStatus.DiscoveryComplete should have comment or be unexported (golint)
  • Line 99: warning: exported method InstallStatus.RecipeAvailable should have comment or be unexported (golint)
  • Line 109: warning: exported method InstallStatus.RecipesAvailable should have comment or be unexported (golint)
  • Line 119: warning: exported method InstallStatus.RecipesSelected should have comment or be unexported (golint)
  • Line 127: warning: exported method InstallStatus.RecipeInstalled should have comment or be unexported (golint)
  • Line 152: warning: exported method InstallStatus.RecipeInstalling should have comment or be unexported (golint)
  • Line 162: warning: exported method InstallStatus.RecipeFailed should have comment or be unexported (golint)
  • Line 172: warning: exported method InstallStatus.RecipeSkipped should have comment or be unexported (golint)
  • Line 182: warning: exported method InstallStatus.InstallComplete should have comment or be unexported (golint)
  • Line 192: warning: exported method InstallStatus.InstallCanceled should have comment or be unexported (golint)
  • Line 224: warning: exported method InstallStatus.SetTargetedInstall should have comment or be unexported (golint)
  • Line 228: warning: exported method InstallStatus.IsTargetedInstall should have comment or be unexported (golint)
  • Line 232: warning: exported method InstallStatus.HostEntityGUID should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/mock_recipe_executor.go
  • Line 9: warning: exported type MockRecipeExecutor should have comment or be unexported (golint)
  • Line 13: warning: exported function NewMockRecipeExecutor should have comment or be unexported (golint)
  • Line 19: warning: exported method MockRecipeExecutor.Prepare should have comment or be unexported (golint)
  • Line 23: warning: exported method MockRecipeExecutor.Execute should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/mock_discoverer.go
  • Line 9: warning: exported type MockDiscoverer should have comment or be unexported (golint)
  • Line 13: warning: exported function NewMockDiscoverer should have comment or be unexported (golint)
  • Line 25: warning: exported method MockDiscoverer.SetOs should have comment or be unexported (golint)
  • Line 29: warning: exported method MockDiscoverer.SetPlatform should have comment or be unexported (golint)
  • Line 33: warning: exported method MockDiscoverer.SetPlatformVersion should have comment or be unexported (golint)
  • Line 37: warning: exported method MockDiscoverer.GetManifest should have comment or be unexported (golint)
  • Line 41: warning: exported method MockDiscoverer.Discover should have comment or be unexported (golint)

• newrelic-cli/internal/output/output.go
  • Line 17: warning: exported const DefaultFormat should have comment or be unexported (golint)
  • Line 18: warning: exported const DefaultPretty should have comment or be unexported (golint)
  • Line 19: warning: exported const DefaultTerminalWidth should have comment or be unexported (golint)
  • Line 22: warning: exported const FormatJSON should have comment (or a comment on this block) or be unexported (golint)
  • Line 58: warning: exported function FormatOptions should have comment or be unexported (golint)
  • Line 68: warning: exported function ParseFormat should have comment or be unexported (golint)
  • Line 78: warning: exported function SetFormat should have comment or be unexported (golint)
  • Line 88: warning: exported function SetPrettyPrint should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/mock_recipe_fetcher.go
  • Line 10: warning: exported type MockRecipeFetcher should have comment or be unexported (golint)
  • Line 24: warning: exported function NewMockRecipeFetcher should have comment or be unexported (golint)
  • Line 32: warning: exported method MockRecipeFetcher.FetchRecipe should have comment or be unexported (golint)
  • Line 44: warning: exported method MockRecipeFetcher.FetchRecipes should have comment or be unexported (golint)
  • Line 49: warning: exported method MockRecipeFetcher.FetchRecommendations should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/process_filterer.go
  • Line 9: warning: exported type ProcessFilterer should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/mock_nerdstorage_client.go
  • Line 5: warning: exported type MockNerdStorageClient should have comment or be unexported (golint)
  • Line 17: warning: exported function NewMockNerdStorageClient should have comment or be unexported (golint)
  • Line 26: warning: exported method MockNerdStorageClient.WriteDocumentWithUserScope should have comment or be unexported (golint)
  • Line 31: warning: exported method MockNerdStorageClient.WriteDocumentWithEntityScope should have comment or be unexported (golint)
  • Line 36: warning: exported method MockNerdStorageClient.WriteDocumentWithAccountScope should have comment or be unexported (golint)

• newrelic-cli/internal/install/validation/mock_recipe_validator.go
  • Line 11: warning: exported type MockRecipeValidator should have comment or be unexported (golint)
  • Line 19: warning: exported function NewMockRecipeValidator should have comment or be unexported (golint)
  • Line 23: warning: exported method MockRecipeValidator.ValidateRecipe should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/manifest_validator.go
  • Line 9: warning: exported type ManifestValidator should have comment or be unexported (golint)
  • Line 13: warning: exported type Validator should have comment or be unexported (golint)
  • Line 35: warning: exported method ManifestValidator.Execute should have comment or be unexported (golint)
  • Line 51: warning: exported method ManifestValidator.FindAllValidationErrors should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/terminal_reporter.go
  • Line 11: warning: exported type TerminalStatusReporter should have comment or be unexported (golint)
  • Line 20: warning: exported method TerminalStatusReporter.RecipeFailed should have comment or be unexported (golint)
  • Line 24: warning: exported method TerminalStatusReporter.RecipeInstalling should have comment or be unexported (golint)
  • Line 28: warning: exported method TerminalStatusReporter.RecipeInstalled should have comment or be unexported (golint)
  • Line 32: warning: exported method TerminalStatusReporter.RecipeSkipped should have comment or be unexported (golint)
  • Line 36: warning: exported method TerminalStatusReporter.RecipeRecommended should have comment or be unexported (golint)
  • Line 40: warning: exported method TerminalStatusReporter.RecipesAvailable should have comment or be unexported (golint)
  • Line 44: warning: exported method TerminalStatusReporter.RecipesSelected should have comment or be unexported (golint)
  • Line 66: warning: exported method TerminalStatusReporter.RecipeAvailable should have comment or be unexported (golint)
  • Line 70: warning: exported method TerminalStatusReporter.InstallComplete should have comment or be unexported (golint)
  • Line 112: warning: exported method TerminalStatusReporter.InstallCanceled should have comment or be unexported (golint)
  • Line 116: warning: exported method TerminalStatusReporter.DiscoveryComplete should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/progress_indicator.go
  • Line 3: warning: exported type ProgressIndicator should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/psutil_discoverer.go
  • Line 16: warning: exported type PSUtilDiscoverer should have comment or be unexported (golint)
  • Line 20: warning: exported function NewPSUtilDiscoverer should have comment or be unexported (golint)
  • Line 28: warning: exported method PSUtilDiscoverer.Discover should have comment or be unexported (golint)

• newrelic-cli/internal/install/license_key_fetcher.go
  • Line 13: warning: comment on exported type ServiceLicenseKeyFetcher should be of the form "ServiceLicenseKeyFetcher ..." (with optional leading article) (golint)
  • Line 18: warning: exported type LicenseKeyFetcher should have comment or be unexported (golint)
  • Line 22: warning: exported function NewServiceLicenseKeyFetcher should have comment or be unexported (golint)
  • Line 30: warning: exported method ServiceLicenseKeyFetcher.FetchLicenseKey should have comment or be unexported (golint)

• newrelic-cli/internal/config/logging.go
  • Line 46: warning: exported function InitFileLogger should have comment or be unexported (golint)
  • Line 69: warning: exported type LogrusFileHook should have comment or be unexported (golint)
  • Line 76: warning: exported function NewLogrusFileHook should have comment or be unexported (golint)
  • Line 87: warning: exported method LogrusFileHook.Fire should have comment or be unexported (golint)
  • Line 103: warning: exported method LogrusFileHook.Levels should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/nerdstorage_client.go
  • Line 7: warning: exported type NerdStorageClient should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/recipe_file.go
  • Line 14: warning: exported type RecipeFileFetcherImpl should have comment or be unexported (golint)
  • Line 19: warning: exported function NewRecipeFileFetcher should have comment or be unexported (golint)
  • Line 34: warning: exported method RecipeFileFetcherImpl.FetchRecipeFile should have comment or be unexported (golint)
  • Line 54: warning: exported method RecipeFileFetcherImpl.LoadRecipeFile should have comment or be unexported (golint)
  • Line 63: warning: exported function NewRecipeFile should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/mock_prompter.go
  • Line 3: warning: exported type MockPrompter should have comment or be unexported (golint)
  • Line 13: warning: exported function NewMockPrompter should have comment or be unexported (golint)
  • Line 17: warning: exported method MockPrompter.PromptYesNo should have comment or be unexported (golint)
  • Line 22: warning: exported method MockPrompter.MultiSelect should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/os_version_validator.go
  • Line 11: warning: exported type OsVersionValidator should have comment or be unexported (golint)
  • Line 23: warning: exported function NewOsVersionValidator should have comment or be unexported (golint)
  • Line 34: warning: exported method OsVersionValidator.Execute should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/mock_failing_recipe_executor.go
  • Line 10: warning: exported type MockFailingRecipeExecutor should have comment or be unexported (golint)
  • Line 14: warning: exported function NewMockFailingRecipeExecutor should have comment or be unexported (golint)
  • Line 20: warning: exported method MockFailingRecipeExecutor.Prepare should have comment or be unexported (golint)
  • Line 24: warning: exported method MockFailingRecipeExecutor.Execute should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/mock_recipe_file_fetcher.go
  • Line 9: warning: exported type MockRecipeFileFetcher should have comment or be unexported (golint)
  • Line 14: warning: exported function NewMockRecipeFileFetcher should have comment or be unexported (golint)
  • Line 21: warning: exported method MockRecipeFileFetcher.FetchRecipeFile should have comment or be unexported (golint)
  • Line 25: warning: exported method MockRecipeFileFetcher.LoadRecipeFile should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/recipe_file_fetcher.go
  • Line 9: warning: exported type RecipeFileFetcher should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/plain_progress_indicator.go
  • Line 9: warning: exported type PlainProgress should have comment or be unexported (golint)
  • Line 12: warning: exported function NewPlainProgress should have comment or be unexported (golint)
  • Line 18: warning: exported method PlainProgress.Start should have comment or be unexported (golint)
  • Line 27: warning: exported method PlainProgress.Success should have comment or be unexported (golint)
  • Line 36: warning: exported method PlainProgress.Fail should have comment or be unexported (golint)
  • Line 45: warning: exported method PlainProgress.Stop should have comment or be unexported (golint)

• newrelic-cli/internal/install/scenario_builder.go
  • Line 14: warning: exported type TestScenario should have comment or be unexported (golint)
  • Line 17: warning: exported const Basic should have comment (or a comment on this block) or be unexported (golint)
  • Line 22: warning: exported var TestScenarios should have comment or be unexported (golint)
  • Line 38: warning: exported function TestScenarioValues should have comment or be unexported (golint)
  • Line 47: warning: exported type ScenarioBuilder should have comment or be unexported (golint)
  • Line 51: warning: exported function NewScenarioBuilder should have comment or be unexported (golint)
  • Line 59: warning: exported method ScenarioBuilder.BuildScenario should have comment or be unexported (golint)
  • Line 70: warning: exported method ScenarioBuilder.Basic should have comment or be unexported (golint)
  • Line 115: warning: exported method ScenarioBuilder.Fail should have comment or be unexported (golint)

• newrelic-cli/internal/install/types/discovery_manifest.go
  • Line 28: warning: exported type MatchedProcess should have comment or be unexported (golint)
  • Line 39: warning: exported method DiscoveryManifest.ConstrainRecipes should have comment or be unexported (golint)

• newrelic-cli/internal/install/mock_license_key_fetcher.go
  • Line 7: warning: exported type MockLicenseKeyFetcher should have comment or be unexported (golint)
  • Line 11: warning: exported function NewMockLicenseKeyFetcher should have comment or be unexported (golint)
  • Line 17: warning: exported method MockLicenseKeyFetcher.FetchLicenseKey should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/mock_status_reporter.go
  • Line 50: warning: exported method MockStatusReporter.RecipeFailed should have comment or be unexported (golint)
  • Line 59: warning: exported method MockStatusReporter.RecipeInstalled should have comment or be unexported (golint)
  • Line 83: warning: exported method MockStatusReporter.RecipeInstalling should have comment or be unexported (golint)
  • Line 92: warning: exported method MockStatusReporter.RecipeRecommended should have comment or be unexported (golint)
  • Line 101: warning: exported method MockStatusReporter.RecipeSkipped should have comment or be unexported (golint)
  • Line 110: warning: exported method MockStatusReporter.RecipeAvailable should have comment or be unexported (golint)
  • Line 119: warning: exported method MockStatusReporter.RecipesAvailable should have comment or be unexported (golint)
  • Line 124: warning: exported method MockStatusReporter.RecipesSelected should have comment or be unexported (golint)
  • Line 129: warning: exported method MockStatusReporter.InstallComplete should have comment or be unexported (golint)
  • Line 134: warning: exported method MockStatusReporter.InstallCanceled should have comment or be unexported (golint)
  • Line 139: warning: exported method MockStatusReporter.DiscoveryComplete should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/mock_empty_discoverer.go
  • Line 3: warning: exported function NewEmptyManifestValidator should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/psutil_process.go
  • Line 7: warning: exported type PSUtilProcess should have comment or be unexported (golint)
  • Line 9: warning: exported method PSUtilProcess.Name should have comment or be unexported (golint)
  • Line 19: warning: exported method PSUtilProcess.Cmdline should have comment or be unexported (golint)
  • Line 29: warning: exported method PSUtilProcess.PID should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/mock_success_link_generator.go
  • Line 5: warning: exported type MockSuccessLinkGenerator should have comment or be unexported (golint)
  • Line 12: warning: exported function NewMockSuccessLinkGenerator should have comment or be unexported (golint)
  • Line 16: warning: exported method MockSuccessLinkGenerator.GenerateExplorerLink should have comment or be unexported (golint)
  • Line 21: warning: exported method MockSuccessLinkGenerator.GenerateEntityLink should have comment or be unexported (golint)
  • Line 26: warning: exported method MockSuccessLinkGenerator.GenerateRedirectURL should have comment or be unexported (golint)

• newrelic-cli/internal/diagnose/config_validator.go
  • Line 5: warning: exported type ConfigValidator should have comment or be unexported (golint)

• newrelic-cli/internal/utils/terraform/hcl_gen.go
  • Line 8: warning: exported type HCLGen should have comment or be unexported (golint)
  • Line 14: warning: exported function NewHCLGen should have comment or be unexported (golint)
  • Line 21: warning: exported method HCLGen.WriteMultilineStringAttribute should have comment or be unexported (golint)
  • Line 25: warning: exported method HCLGen.WriteMultilineStringAttributeIfNotEmpty should have comment or be unexported (golint)
  • Line 31: warning: exported method HCLGen.WriteStringAttribute should have comment or be unexported (golint)
  • Line 35: warning: exported method HCLGen.WriteStringAttributeIfNotEmpty should have comment or be unexported (golint)
  • Line 41: warning: exported method HCLGen.WriteStringSliceAttribute should have comment or be unexported (golint)
  • Line 45: warning: exported method HCLGen.WriteStringSliceAttributeIfNotEmpty should have comment or be unexported (golint)
  • Line 51: warning: exported method HCLGen.WriteIntAttribute should have comment or be unexported (golint)
  • Line 55: warning: exported method HCLGen.WriteIntAttributeIfNotZero should have comment or be unexported (golint)
  • Line 61: warning: exported method HCLGen.WriteBlock should have comment or be unexported (golint)

• newrelic-cli/internal/output/config.go
  • Line 37: warning: exported type ConfigOption should have comment or be unexported (golint)
  • Line 39: warning: exported function ConfigFormat should have comment or be unexported (golint)
  • Line 46: warning: exported function ConfigPrettyPrint should have comment or be unexported (golint)

• newrelic-cli/internal/utils/terraform/dashboard.go
  • Line 30: warning: exported type DashboardWidgetRawConfiguration should have comment or be unexported (golint)
  • Line 40: warning: exported type DashboardWidgetFacet should have comment or be unexported (golint)
  • Line 44: warning: exported type DashboardWidgetNRQLQuery should have comment or be unexported (golint)
  • Line 49: warning: exported type DashboardWidgetLegend should have comment or be unexported (golint)
  • Line 53: warning: exported type DashboardWidgetYAxisLeft should have comment or be unexported (golint)
  • Line 57: warning: exported function GenerateDashboardHCL should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/line_capture_buffer.go
  • Line 7: warning: exported type LineCaptureBuffer should have comment or be unexported (golint)
  • Line 13: warning: exported function NewLineCaptureBuffer should have comment or be unexported (golint)
  • Line 37: warning: exported method LineCaptureBuffer.Current should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/spinner.go
  • Line 22: warning: exported type Spinner should have comment or be unexported (golint)
  • Line 26: warning: exported function NewSpinner should have comment or be unexported (golint)
  • Line 33: warning: exported method Spinner.Start should have comment or be unexported (golint)
  • Line 45: warning: exported method Spinner.Stop should have comment or be unexported (golint)
  • Line 53: warning: exported method Spinner.Fail should have comment or be unexported (golint)
  • Line 58: warning: exported method Spinner.Success should have comment or be unexported (golint)

• newrelic-cli/internal/decode/command.go
  • Line 7: warning: exported var Command should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/go_task_recipe_executor.go
  • Line 33: warning: exported method GoTaskRecipeExecutor.Prepare should have comment or be unexported (golint)
  • Line 68: warning: exported method GoTaskRecipeExecutor.Execute should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/success_link_generator.go
  • Line 12: warning: exported type SuccessLinkGenerator should have comment or be unexported (golint)
  • Line 18: warning: exported type ConcreteSuccessLinkGenerator should have comment or be unexported (golint)
  • Line 30: warning: exported function NewConcreteSuccessLinkGenerator should have comment or be unexported (golint)
  • Line 34: warning: exported method ConcreteSuccessLinkGenerator.GenerateExplorerLink should have comment or be unexported (golint)
  • Line 38: warning: exported method ConcreteSuccessLinkGenerator.GenerateEntityLink should have comment or be unexported (golint)

• newrelic-cli/internal/utils/retry.go
  • Line 5: warning: exported type Retry should have comment or be unexported (golint)
  • Line 11: warning: exported function NewRetry should have comment or be unexported (golint)
  • Line 19: warning: exported method Retry.ExecWithRetries should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/nerdgraph_client.go
  • Line 5: warning: exported type NerdGraphClient should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/prompter.go
  • Line 3: warning: exported type Prompter should have comment or be unexported (golint)

• newrelic-cli/internal/install/types/errors.go
  • Line 16: warning: exported type GoTaskError should have comment or be unexported (golint)
  • Line 22: warning: comment on exported type GoTaskGeneralError should be of the form "GoTaskGeneralError ..." (with optional leading article) (golint)
  • Line 32: warning: exported method GoTaskGeneralError.SetError should have comment or be unexported (golint)
  • Line 36: warning: exported method GoTaskGeneralError.TaskPath should have comment or be unexported (golint)
  • Line 40: warning: exported function NewGoTaskGeneralError should have comment or be unexported (golint)
  • Line 71: warning: exported function NewNonZeroExitCode should have comment or be unexported (golint)
  • Line 86: warning: comment on exported var ErrorFetchingLicenseKey should be of the form "ErrorFetchingLicenseKey ..." (golint)
  • Line 88: warning: exported var ErrorFetchingInsightsInsertKey should have comment or be unexported (golint)

• newrelic-cli/internal/apiaccess/command.go
  • Line 15: warning: exported var Command should have comment or be unexported (golint)

• newrelic-cli/internal/install/install_context.go
  • Line 3: warning: comment on exported type InstallerContext should be of the form "InstallerContext ..." (with optional leading article) (golint)
  • Line 17: warning: exported method InstallerContext.ShouldRunDiscovery should have comment or be unexported (golint)
  • Line 21: warning: exported method InstallerContext.ShouldInstallInfraAgent should have comment or be unexported (golint)
  • Line 25: warning: exported method InstallerContext.ShouldInstallLogging should have comment or be unexported (golint)
  • Line 29: warning: exported method InstallerContext.ShouldInstallIntegrations should have comment or be unexported (golint)
  • Line 33: warning: exported method InstallerContext.ShouldInstallApm should have comment or be unexported (golint)
  • Line 37: warning: exported method InstallerContext.RecipePathsProvided should have comment or be unexported (golint)
  • Line 41: warning: exported method InstallerContext.RecipeNamesProvided should have comment or be unexported (golint)
  • Line 45: warning: exported method InstallerContext.RecipesProvided should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipe_installer.go
  • Line 22: warning: exported type RecipeInstaller should have comment or be unexported (golint)
  • Line 38: warning: exported function NewRecipeInstaller should have comment or be unexported (golint)
  • Line 90: warning: exported method RecipeInstaller.Install should have comment or be unexported (golint)

• newrelic-cli/internal/install/execution/nerdstorage_status_reporter.go
  • Line 37: warning: exported method NerdstorageStatusReporter.RecipesSelected should have comment or be unexported (golint)
  • Line 47: warning: exported method NerdstorageStatusReporter.RecipeFailed should have comment or be unexported (golint)
  • Line 51: warning: exported method NerdstorageStatusReporter.RecipeInstalling should have comment or be unexported (golint)
  • Line 55: warning: exported method NerdstorageStatusReporter.RecipeInstalled should have comment or be unexported (golint)
  • Line 59: warning: exported method NerdstorageStatusReporter.RecipeRecommended should have comment or be unexported (golint)
  • Line 63: warning: exported method NerdstorageStatusReporter.RecipeSkipped should have comment or be unexported (golint)
  • Line 67: warning: exported method NerdstorageStatusReporter.InstallComplete should have comment or be unexported (golint)
  • Line 71: warning: exported method NerdstorageStatusReporter.InstallCanceled should have comment or be unexported (golint)
  • Line 75: warning: exported method NerdstorageStatusReporter.DiscoveryComplete should have comment or be unexported (golint)

• newrelic-cli/internal/utils/nrdb_client.go
  • Line 9: warning: exported type NRDBClient should have comment or be unexported (golint)

• newrelic-cli/internal/install/recipes/local_recipe_fetcher.go
  • Line 16: warning: exported type LocalRecipeFetcher should have comment or be unexported (golint)
  • Line 20: warning: exported method LocalRecipeFetcher.FetchRecipe should have comment or be unexported (golint)
  • Line 35: warning: exported method LocalRecipeFetcher.FetchRecommendations should have comment or be unexported (golint)
  • Line 44: warning: exported method LocalRecipeFetcher.FetchRecipes should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/mock_progress_indicator.go
  • Line 3: warning: exported type MockProgressIndicator should have comment or be unexported (golint)
  • Line 5: warning: exported function NewMockProgressIndicator should have comment or be unexported (golint)
  • Line 9: warning: exported method MockProgressIndicator.Fail should have comment or be unexported (golint)
  • Line 12: warning: exported method MockProgressIndicator.Success should have comment or be unexported (golint)
  • Line 15: warning: exported method MockProgressIndicator.Start should have comment or be unexported (golint)
  • Line 18: warning: exported method MockProgressIndicator.Stop should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/mock_file_filterer.go
  • Line 22: warning: exported method MockFileFilterer.Filter should have comment or be unexported (golint)

• newrelic-cli/internal/install/ux/prompt_ui_prompter.go
  • Line 11: warning: exported type PromptUIPrompter should have comment or be unexported (golint)
  • Line 13: warning: exported function NewPromptUIPrompter should have comment or be unexported (golint)
  • Line 17: warning: exported method PromptUIPrompter.PromptYesNo should have comment or be unexported (golint)
  • Line 33: warning: exported method PromptUIPrompter.MultiSelect should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/os_validator.go
  • Line 10: warning: exported type OsValidator should have comment or be unexported (golint)
  • Line 19: warning: exported function NewOsValidator should have comment or be unexported (golint)
  • Line 25: warning: exported method OsValidator.Execute should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/noop_process_filterer.go
  • Line 9: warning: exported type NoOpProcessFilterer should have comment or be unexported (golint)
  • Line 11: warning: exported function NewNoOpProcessFilterer should have comment or be unexported (golint)

• newrelic-cli/internal/install/discovery/regex_process_filterer.go
  • Line 14: warning: exported type RegexProcessFilterer should have comment or be unexported (golint)
  • Line 18: warning: exported function NewRegexProcessFilterer should have comment or be unexported (golint)

• newrelic-cli/internal/diagnose/concrete_config_validator.go
  • Line 21: warning: exported const DefaultPostRetryDelaySec should have comment (or a comment on this block) or be unexported (golint)
  • Line 26: warning: exported type ConcreteConfigValidator should have comment or be unexported (golint)
  • Line 34: warning: exported type ValidationTracerEvent should have comment or be unexported (golint)
  • Line 41: warning: exported function NewConcreteConfigValidator should have comment or be unexported (golint)
  • Line 54: warning: exported method ConcreteConfigValidator.ValidateConfig should have comment or be unexported (golint)

• newrelic-cli/internal/diagnose/mock_config_validator.go
  • Line 5: warning: exported type MockConfigValidator should have comment or be unexported (golint)
  • Line 7: warning: exported function NewMockConfigValidator should have comment or be unexported (golint)
  • Line 11: warning: exported method MockConfigValidator.ValidateConfig should have comment or be unexported (golint)

• newrelic-cli/internal/utils/utils.go
  • Line 20: warning: exported var SignalCtx should have comment or be unexported (golint)
  • Line 35: warning: exported type StructToMapCallback should have comment or be unexported (golint)
  • Line 37: warning: exported function StructToMap should have comment or be unexported (golint)
  • Line 126: warning: comment on exported function StdinExists should be of the form "StdinExists ..." (golint)

• newrelic-cli/internal/install/types/recipe.go
  • Line 12: warning: exported const InfraAgentRecipeName should have comment (or a comment on this block) or be unexported (golint)
  • Line 17: warning: exported var RecipeVariables should have comment or be unexported (golint)
  • Line 23: warning: comment on exported method OpenInstallationRecipe.UnmarshalYAML should be of the form "UnmarshalYAML ..." (golint)
  • Line 334: warning: exported method OpenInstallationRecipe.PostInstallMessage should have comment or be unexported (golint)
  • Line 342: warning: exported method OpenInstallationRecipe.PreInstallMessage should have comment or be unexported (golint)
  • Line 356: warning: exported method OpenInstallationRecipe.IsApm should have comment or be unexported (golint)
  • Line 360: warning: exported method OpenInstallationRecipe.HasHostTargetType should have comment or be unexported (golint)
  • Line 364: warning: exported method OpenInstallationRecipe.HasApplicationTargetType should have comment or be unexported (golint)
  • Line 368: warning: exported method OpenInstallationRecipe.HasKeyword should have comment or be unexported (golint)
  • Line 382: warning: exported method OpenInstallationRecipe.HasTargetType should have comment or be unexported (golint)

• newrelic-cli/internal/install/validation/mock_nrdb_client.go
  • Line 10: warning: exported type MockNRDBClient should have comment or be unexported (golint)
  • Line 16: warning: exported function NewMockNRDBClient should have comment or be unexported (golint)
  • Line 24: warning: exported method MockNRDBClient.QueryWithContext should have comment or be unexported (golint)
  • Line 36: warning: exported method MockNRDBClient.ThrowError should have comment or be unexported (golint)
  • Line 40: warning: exported method MockNRDBClient.ReturnResultsAfterNAttempts should have comment or be unexported (golint)
  • Line 50: warning: exported method MockNRDBClient.Attempts should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!