When config file isn't found, this shouldn't be fatal

sgsullivan · sgsullivan · commit 97d0e47b607c · 2020-12-31T11:53:17.000-05:00
new installs wont have this before they add contexts.
diff --git a/config/core.go b/config/core.go
@@ -34,8 +34,13 @@ func InitConfig() (vp *viper.Viper, err error) {
 	}
 
 	vp.AutomaticEnv()
-	if err := vp.ReadInConfig(); err != nil {
-		utils.PrintYellow("no config\n")
+	if err = vp.ReadInConfig(); err != nil {
+		if _, notFound := err.(viper.ConfigFileNotFoundError); notFound {
+			err = nil
+			return
+		}
+		utils.PrintYellow("error reading config: %s\n", err)
+		return
 	}
 
 	if UseContextArg != "" {

Original file line number	Diff line number	Diff line change
`@@ -34,8 +34,13 @@ func InitConfig() (vp *viper.Viper, err error) {`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`vp.AutomaticEnv()`
`37`		`- if err := vp.ReadInConfig(); err != nil {`
`38`		`- utils.PrintYellow("no config\n")`
	`37`	`+ if err = vp.ReadInConfig(); err != nil {`
	`38`	`+ if _, notFound := err.(viper.ConfigFileNotFoundError); notFound {`
	`39`	`+ err = nil`
	`40`	`+ return`
	`41`	`+ }`
	`42`	`+ utils.PrintYellow("error reading config: %s\n", err)`
	`43`	`+ return`
`39`	`44`	`}`
`40`	`45`
`41`	`46`	`if UseContextArg != "" {`