Merge pull request ActiveState#3362 from ActiveState/mitchell/dx-2911

Implement error interface instead of including error object field in struct.

Author: mitchell-as

internal/runners/initialize/init.go

@@ -78,10 +78,13 @@ var errNoOwner = errs.New("Could not find organization")
 var errNoLanguage = errs.New("No language specified")
 
 type errUnrecognizedLanguage struct {
-	error
 	Name string
 }
 
+func (e errUnrecognizedLanguage) Error() string {
+	return fmt.Sprintf("unrecognized language: %s", e.Name)
+}
+
 // New returns a prepared ptr to Initialize instance.
 func New(prime primeable) *Initialize {
 	return &Initialize{prime.Auth(), prime.Config(), prime.Output(), prime.Analytics(), prime.SvcModel()}
@@ -183,12 +186,12 @@ func (r *Initialize) Run(params *RunParams) (rerr error) {
 
 	// Require 'python', 'python@3', or 'python@2' instead of 'python3' or 'python2'.
 	if languageName == language.Python3.String() || languageName == language.Python2.String() {
-		return &errUnrecognizedLanguage{Name: languageName}
+		return &errUnrecognizedLanguage{languageName}
 	}
 
 	lang := language.MakeByNameAndVersion(languageName, languageVersion)
 	if !lang.Recognized() {
-		return &errUnrecognizedLanguage{Name: languageName}
+		return &errUnrecognizedLanguage{languageName}
 	}
 
 	version, err := deriveVersion(lang, languageVersion, r.auth)
@@ -327,7 +330,7 @@ func getKnownVersions(lang language.Language, auth *authentication.Auth) ([]stri
 	}
 
 	if len(pkgs) == 0 {
-		return nil, &errUnrecognizedLanguage{Name: lang.Requirement()}
+		return nil, &errUnrecognizedLanguage{lang.Requirement()}
 	}
 
 	knownVersions := make([]string, len(pkgs))