further improvements to error message

debricked · Dec 12, 2023 · 78976e8 · 78976e8
1 parent 146ee6d
commit 78976e8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/internal/resolution/pm/pip/job.go b/internal/resolution/pm/pip/job.go
@@ -164,7 +164,7 @@ func (j *Job) handleInstallError(cmdErr job.IError) {
 				[]string{
 					"Failed to find a version that satisfies the requirement for python dependency ",
 					dependencyName,
-					". This could mean that the specified version or version does not exist.\n" + util.InstalLPrivateDependencyMessage,
+					". This could mean that the package or version does not exist.\n" + util.InstalLPrivateDependencyMessage,
 				}, ""),
 		)
 	}

diff --git a/internal/resolution/pm/util/error.go b/internal/resolution/pm/util/error.go
@@ -45,7 +45,7 @@ func NewPMJobError(err string) *PMJobError {
 	return &PMJobError{
 		err:    err,
 		cmd:    "",
-		doc:    "No specific documentation for this problem yet, please create an issue here: https://github.com/debricked/cli/issues",
+		doc:    "No specific documentation for this problem yet. If you would like this message to more informative for this error, please create an issue here: https://github.com/debricked/cli/issues",
 		status: "",
 	}
 }
diff --git a/internal/resolution/pm/util/error_test.go b/internal/resolution/pm/util/error_test.go
@@ -61,7 +61,7 @@ func TestPMJobErrorSetDocumentation(t *testing.T) {
 	jobError := NewPMJobError("")
 	assert.Equal(
 		t,
-		string("No specific documentation for this problem yet, please create an issue here: https://github.com/debricked/cli/issues\n"),
+		string("No specific documentation for this problem yet. If you would like this message to more informative for this error, please create an issue here: https://github.com/debricked/cli/issues\n"),
 		jobError.Documentation(),
 	)
 	jobError.SetDocumentation("documentation")