Skip to content

Commit

Permalink
Removing TODO about moving sendMetrics into main() function
Browse files Browse the repository at this point in the history
  • Loading branch information
andygrunwald committed Sep 16, 2024
1 parent 077bf7b commit efb5a5b
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions cmd/sync.go
Original file line number Diff line number Diff line change
Expand Up @@ -532,8 +532,7 @@ func runSyncCommandLogic(p *tea.Program, logger *slog.Logger, config *syncComman
})
}

// TODO Move send metrics into main method
// Catch result, see https://github.com/charmbracelet/bubbletea/blob/main/examples/result/main.go
// Send metrics to Buchhalter API
alwaysSendMetrics := viper.GetBool("buchhalter_always_send_metrics")
if !developmentMode && alwaysSendMetrics {
logger.Info("Sending usage metrics to Buchhalter API", "always_send_metrics", alwaysSendMetrics, "development_mode", developmentMode)
Expand Down

0 comments on commit efb5a5b

Please sign in to comment.