From e15a2aa0d53a632c54329f1817d91871ec90d573 Mon Sep 17 00:00:00 2001 From: Anatoli Babenia Date: Sat, 22 Feb 2020 07:10:26 +0300 Subject: [PATCH] Make `errcheck` happy --- cmd/root.go | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/cmd/root.go b/cmd/root.go index 3ce286a..707c556 100644 --- a/cmd/root.go +++ b/cmd/root.go @@ -66,6 +66,8 @@ func initCli() { // initConfig reads in config file and ENV variables if set. func initConfig() { + var err error + viper.SetDefault("log.level", log.InfoLevel.String()) viper.SetDefault("log.path", "./dive.log") viper.SetDefault("log.enabled", false) @@ -97,7 +99,11 @@ func initConfig() { viper.SetDefault("container-engine", "docker") - viper.BindPFlag("source", rootCmd.PersistentFlags().Lookup("source")) + err = viper.BindPFlag("source", rootCmd.PersistentFlags().Lookup("source")) + if err != nil { + fmt.Println(err) + os.Exit(1) + } viper.SetEnvPrefix("DIVE") // replace all - with _ when looking for matching environment variables @@ -112,7 +118,7 @@ func initConfig() { } else { viper.SetConfigFile(cfgFile) } - err := viper.ReadInConfig() + err = viper.ReadInConfig() if err == nil { fmt.Println("Using config file:", viper.ConfigFileUsed()) } else if cfgFile != "" {