fix: do not use global variable

gobrew.go

@@ -55,8 +55,6 @@ type GoBrew struct {
 	downloadsDir  string
 }
 
-var gb GoBrew
-
 // NewGoBrew instance
 func NewGoBrew() GoBrew {
 	homeDir, err := os.UserHomeDir()
@@ -72,6 +70,7 @@ func NewGoBrew() GoBrew {
 }
 
 func NewGoBrewDirectory(homeDir string) GoBrew {
+	gb := GoBrew{}
 	gb.homeDir = homeDir
 
 	gb.installDir = filepath.Join(gb.homeDir, goBrewDir)
@@ -281,9 +280,7 @@ func (gb *GoBrew) ListRemoteVersions(print bool) map[string][]string {
 	tags := gb.getGolangVersions()
 
 	var versions []string
-	for _, tag := range tags {
-		versions = append(versions, tag)
-	}
+	versions = append(versions, tags...)
 
 	return gb.getGroupedVersion(versions, print)
 }

gobrew_test.go

@@ -173,7 +173,7 @@ func TestUpgrade(t *testing.T) {
 	binaryFile := filepath.Join(binaryDir, baseName)
 
 	if oldFile, err := os.Create(binaryFile); err == nil {
-		// on tests we have to close the file to avoid an error on os.Rename
+		// on tests, we have to close the file to avoid an error on os.Rename
 		_ = oldFile.Close()
 	}