Skip to content

Commit

Permalink
root cmd: Remove TODO about writing full configuration file
Browse files Browse the repository at this point in the history
  • Loading branch information
andygrunwald committed Sep 16, 2024
1 parent e39b333 commit 49e5a7a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,9 +134,10 @@ func initConfig() {
os.Exit(1)
}

// TODO We should check if we really want to write the full config here
// Write the full config into the configuration file
// This was introduced to keep the value `buchhalter_always_send_metrics` persistent in the configuration file
// This can turn into problems later on, because we don't need all configuration values in the configuration file persistent
// Right now we don't have such a problem, hence we keep it as is.
err = viper.WriteConfigAs(configFile)
if err != nil {
fmt.Println("Error creating config file:", err)
Expand Down

0 comments on commit 49e5a7a

Please sign in to comment.