fix: add missing logger.Sync()

build/info.go

@@ -5,7 +5,6 @@ import (
 	"runtime/debug"
 	"time"
 
-	"github.com/JulesMike/spoty/logger"
 	"go.uber.org/fx"
 )
 
@@ -22,7 +21,7 @@ type Info struct {
 }
 
 // New returns a new instance of Info.
-func New(logger *logger.Logger) (*Info, error) {
+func New() (*Info, error) {
 	bi, ok := debug.ReadBuildInfo()
 	if !ok {
 		return nil, fmt.Errorf("failed to read build info")
@@ -43,7 +42,7 @@ func New(logger *logger.Logger) (*Info, error) {
 		case "vcs.time":
 			hash, err := time.Parse(time.RFC3339, kv.Value)
 			if err != nil {
-				logger.Warnw("failed to parse vcs.time", "error", err, "value", kv.Value)
+				return nil, fmt.Errorf("failed to parse vcs.time: %w", err)
 			}
 
 			info.LastCommit = hash

cmd/version.go

@@ -5,8 +5,6 @@ import (
 	"os"
 
 	"github.com/JulesMike/spoty/build"
-	"github.com/JulesMike/spoty/config"
-	"github.com/JulesMike/spoty/logger"
 	"github.com/spf13/cobra"
 )
 
@@ -15,17 +13,12 @@ var versionCmd = &cobra.Command{
 	Use:   "version",
 	Short: "Return the build information",
 	Run: func(cmd *cobra.Command, args []string) {
-		logger, err := logger.New(&config.Config{})
+		info, err := build.New()
 		if err != nil {
-			fmt.Printf("failed to start logger: %v", err)
+			fmt.Print(err)
 			os.Exit(1) //nolint:revive
 		}
 
-		info, err := build.New(logger)
-		if err != nil {
-			logger.Fatal(err)
-		}
-
 		fmt.Printf("Revision: %v\n", info.Revision)
 		fmt.Printf("Last Commit: %v\n", info.LastCommit)
 		fmt.Printf("Dirty Build: %v\n", info.DirtyBuild)

logger/logger.go

@@ -1,6 +1,7 @@
 package logger
 
 import (
+	"context"
 	"fmt"
 
 	"github.com/JulesMike/spoty/config"
@@ -20,7 +21,7 @@ type Logger struct {
 }
 
 // New creates a new Logger.
-func New(cfg *config.Config) (*Logger, error) {
+func New(lc fx.Lifecycle, cfg *config.Config) (*Logger, error) {
 	var (
 		logger *zap.Logger
 		err    error
@@ -35,6 +36,14 @@ func New(cfg *config.Config) (*Logger, error) {
 		return nil, fmt.Errorf("failed to create logger: %w", err)
 	}
 
+	lc.Append(fx.Hook{
+		OnStop: func(_ context.Context) error {
+			logger.Sync()
+
+			return nil
+		},
+	})
+
 	otellogger := otelzap.New(logger)
 
 	return &Logger{otellogger.Sugar()}, nil