Merge pull request cli#387 from ahmedelgabri/master

Nate Smith · web-flow · commit 65a7d0520547 · 2020-02-13T13:14:41.000-06:00
Make sure editor is set properly &amp; not overridden later
diff --git a/pkg/surveyext/editor.go b/pkg/surveyext/editor.go
@@ -25,14 +25,11 @@ var (
 func init() {
 	if runtime.GOOS == "windows" {
 		editor = "notepad"
-	}
-	if g := os.Getenv("GIT_EDITOR"); g != "" {
+	} else if g := os.Getenv("GIT_EDITOR"); g != "" {
 		editor = g
-	}
-	if v := os.Getenv("VISUAL"); v != "" {
+	} else if v := os.Getenv("VISUAL"); v != "" {
 		editor = v
-	}
-	if e := os.Getenv("EDITOR"); e != "" {
+	} else if e := os.Getenv("EDITOR"); e != "" {
 		editor = e
 	}
 }
