refactor(logging): Centralize logging configuration

Consolidate logging setup by passing `GitVersionOptions` directly to `LoggingEnricher`, removing redundant configuration logic from `GitVersionExecutor`.

Author: arturcic

src/GitVersion.App/GitVersionApp.cs

@@ -1,4 +1,5 @@
 using GitVersion.Extensions;
+using GitVersion.Logging;
 
 namespace GitVersion;
 
@@ -16,6 +17,7 @@ public Task RunAsync(CancellationToken _)
         try
         {
             var gitVersionOptions = this.options.Value;
+            LoggingEnricher.Configure(gitVersionOptions);
 
             if (gitVersionOptions.IsHelp || gitVersionOptions.IsVersion)
             {

src/GitVersion.App/GitVersionExecutor.cs

@@ -3,7 +3,6 @@
 using GitVersion.Extensions;
 using GitVersion.Git;
 using GitVersion.Helpers;
-using GitVersion.Logging;
 
 namespace GitVersion;
 
@@ -115,8 +114,6 @@ private void Initialize(GitVersionOptions gitVersionOptions)
             gitVersionOptions.Settings.NoCache = true;
         }
 
-        ConfigureLogging(gitVersionOptions);
-
         var workingDirectory = gitVersionOptions.WorkingDirectory;
         if (gitVersionOptions.Diag)
         {
@@ -146,11 +143,4 @@ private bool VerifyAndDisplayConfiguration(GitVersionOptions gitVersionOptions)
         this.console.WriteLine(configurationString);
         return true;
     }
-
-    private static void ConfigureLogging(GitVersionOptions gitVersionOptions)
-    {
-        var enableConsoleOutput = gitVersionOptions.Output.Contains(OutputType.BuildServer)
-                                  || string.Equals(gitVersionOptions.LogFilePath, "console", StringComparison.OrdinalIgnoreCase);
-        LoggingEnricher.Configure(gitVersionOptions.LogFilePath, gitVersionOptions.Verbosity, enableConsoleOutput);
-    }
 }

src/GitVersion.Core/Logging/LoggingEnricher.cs

@@ -47,31 +47,16 @@ public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
         logEvent.AddPropertyIfAbsent(logFilePathProp);
     }
 
-    /// <summary>
-    /// Configures the logging enricher with the specified log file path and verbosity.
-    /// </summary>
-    /// <param name="logFile">The log file path, or null to disable file logging.</param>
-    /// <param name="verbosity">The verbosity level.</param>
-    /// <param name="enableConsoleOutput">Whether to enable console output. When false, console output is suppressed.</param>
-    public static void Configure(string? logFile, Verbosity verbosity, bool enableConsoleOutput = true)
+    public static void Configure(GitVersionOptions gitVersionOptions)
     {
-        if (!string.IsNullOrWhiteSpace(logFile))
-            logFilePath = logFile;
-        LogLevelSwitch.MinimumLevel = GetLevelForVerbosity(verbosity);
-        IsConsoleEnabled = enableConsoleOutput;
-    }
+        var enableConsoleOutput = gitVersionOptions.Output.Contains(OutputType.BuildServer)
+                                  || string.Equals(gitVersionOptions.LogFilePath, "console", StringComparison.OrdinalIgnoreCase);
 
-    /// <summary>
-    /// Configures the logging enricher with the specified log file path and log level.
-    /// </summary>
-    /// <param name="logFile">The log file path, or null to disable file logging.</param>
-    /// <param name="logLevel">The Microsoft.Extensions.Logging log level.</param>
-    /// <param name="enableConsoleOutput">Whether to enable console output. When false, console output is suppressed.</param>
-    public static void Configure(string? logFile, LogLevel logLevel, bool enableConsoleOutput = true)
-    {
-        if (!string.IsNullOrWhiteSpace(logFile))
-            logFilePath = logFile;
-        LogLevelSwitch.MinimumLevel = GetLevelForLogLevel(logLevel);
+        if (!string.IsNullOrWhiteSpace(gitVersionOptions.LogFilePath))
+        {
+            logFilePath = gitVersionOptions.LogFilePath;
+        }
+        LogLevelSwitch.MinimumLevel = GetLevelForVerbosity(gitVersionOptions.Verbosity);
         IsConsoleEnabled = enableConsoleOutput;
     }
 
@@ -85,15 +70,4 @@ public static void Configure(string? logFile, LogLevel logLevel, bool enableCons
         { Verbosity.Minimal, LogEventLevel.Warning },
         { Verbosity.Quiet, LogEventLevel.Error }
     };
-
-    private static LogEventLevel GetLevelForLogLevel(LogLevel logLevel) => logLevel switch
-    {
-        LogLevel.Trace => LogEventLevel.Verbose,
-        LogLevel.Debug => LogEventLevel.Debug,
-        LogLevel.Information => LogEventLevel.Information,
-        LogLevel.Warning => LogEventLevel.Warning,
-        LogLevel.Error => LogEventLevel.Error,
-        LogLevel.Critical or LogLevel.None => LogEventLevel.Fatal,
-        _ => LogEventLevel.Information
-    };
 }