Revert "test with boolp, does not respect env var priority"

This reverts commit 62b1e1f519b704d14f2d4e4fed555d6da6edd26a.

Author: robinovitch61

cmd/root.go

@@ -12,7 +12,6 @@ import (
 
 type arg struct {
 	cliShort, cliLong, cfgFileEnvVar, description string
-	isBool, defaultIfBool                         bool
 }
 
 var (
@@ -158,8 +157,6 @@ var (
 	}
 	startCompactArg = arg{
 		cliLong:       "compact-header",
-		isBool:        true,
-		defaultIfBool: false,
 		cfgFileEnvVar: "wander_compact_header",
 		description:   `If "true", start with compact header. Default "false"`,
 	}
@@ -225,11 +222,7 @@ func init() {
 		startAllTasksView,
 		compactTablesArg,
 	} {
-		if c.isBool {
-			rootCmd.PersistentFlags().BoolP(c.cliLong, c.cliShort, c.defaultIfBool, c.description)
-		} else {
-			rootCmd.PersistentFlags().StringP(c.cliLong, c.cliShort, "", c.description)
-		}
+		rootCmd.PersistentFlags().StringP(c.cliLong, c.cliShort, "", c.description)
 		viper.BindPFlag(c.cliLong, rootCmd.PersistentFlags().Lookup(c.cfgFileEnvVar))
 	}