Skip to content

Commit

Permalink
Let user disable version update reminders
Browse files Browse the repository at this point in the history
  • Loading branch information
camerondurham committed May 28, 2021
1 parent bcd2296 commit 025bfe1
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 13 deletions.
35 changes: 22 additions & 13 deletions cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,7 @@ var rootCmd = &cobra.Command{

func Execute() {

// TODO: only check if user didn't specify they don't want upgrade reminders
latestVersion, err := util.GetLatestVersion(util.GetRequest)
if err != nil {
util.DebugPrint(fmt.Sprintf("ignoring version check since error occured when retrieving latest version: %v\n", err))
} else if version.PkgVersion != "" && latestVersion != version.PkgVersion {
fmt.Printf(
"\tA new version of ch is available!\n"+
"\tYou are running version %s but the latest version is %s.\n"+
"\tSee %s instructions on upgrading.\n",
version.PkgVersion,
latestVersion,
RepositoryUrl)
}
checkCliVersion()

if err := rootCmd.Execute(); err != nil {
fmt.Println(err)
Expand Down Expand Up @@ -79,3 +67,24 @@ func initConfig() {
util.DebugPrint(fmt.Sprint("Using config file:", viper.ConfigFileUsed()))
}
}

func checkCliVersion() {
opts, err := util.GetConfigOpts()

if err != nil {
util.DebugPrint(fmt.Sprintf("error reading environment file: %v", err))
} else if v, ok := opts["version_check"]; !ok || v == "true" {
latestVersion, err := util.GetLatestVersion(util.GetRequest)
if err != nil {
util.DebugPrint(fmt.Sprintf("ignoring version check since error occured when retrieving latest version: %v\n", err))
} else if version.PkgVersion != "" && latestVersion != version.PkgVersion {
fmt.Printf(
"\tA new version of ch is available!\n"+
"\tYou are running version %s but the latest version is %s.\n"+
"\tSee %s instructions on upgrading.\n",
version.PkgVersion,
latestVersion,
RepositoryUrl)
}
}
}
11 changes: 11 additions & 0 deletions cmd/util/cli.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ func (cli *Cli) Containers() map[string]*ContainerOpts {
return envs
}

// GetEnvs retrieves the environments managed by ch from ~/.ch.yaml
func GetEnvs() (envs map[string]*ContainerOpts, err error) {
if !viper.IsSet("envs") {
return nil, ErrDoesNotExist
Expand All @@ -101,6 +102,16 @@ func GetEnvs() (envs map[string]*ContainerOpts, err error) {
return
}

// GetConfigOpts retrieves optional settings from ~/.ch.yaml
func GetConfigOpts() (opts map[string]string, err error) {
if !viper.IsSet("envs") {
return nil, ErrDoesNotExist
}
opts = make(map[string]string)
err = viper.UnmarshalKey("opts", &opts)
return
}

func PrintEnvNotFoundMsg(envName string) {
fmt.Printf("no such environment: %v\n", envName)
envs, err := GetEnvs()
Expand Down

0 comments on commit 025bfe1

Please sign in to comment.