WIP: Migrate core files from ILog to ILogger - partial migration

Co-authored-by: arturcic <[email protected]>

Author: Copilot

src/GitVersion.Configuration/ConfigurationFileLocator.cs

@@ -1,14 +1,14 @@
 using System.IO.Abstractions;
 using GitVersion.Extensions;
 using GitVersion.Helpers;
-using GitVersion.Logging;
+using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
 
 namespace GitVersion.Configuration;
 
 internal class ConfigurationFileLocator(
     IFileSystem fileSystem,
-    ILog log,
+    ILogger<ConfigurationFileLocator> logger,
     IOptions<GitVersionOptions> options)
     : IConfigurationFileLocator
 {
@@ -26,7 +26,7 @@ internal class ConfigurationFileLocator(
     ];
 
     private readonly IFileSystem fileSystem = fileSystem.NotNull();
-    private readonly ILog log = log.NotNull();
+    private readonly ILogger<ConfigurationFileLocator> logger = log.NotNull();
     private readonly IOptions<GitVersionOptions> options = options.NotNull();
 
     private string? ConfigurationFile => options.Value.ConfigurationInfo.ConfigurationFile;
@@ -43,7 +43,7 @@ public void Verify(string? workingDirectory, string? projectRootDirectory)
         var customConfigurationFile = GetCustomConfigurationFilePathIfEligable(directoryPath);
         if (!string.IsNullOrWhiteSpace(customConfigurationFile))
         {
-            this.log.Info($"Found configuration file at '{customConfigurationFile}'");
+            this.logger.LogInformation($"Found configuration file at '{customConfigurationFile}'");
             return customConfigurationFile;
         }
 
@@ -55,10 +55,10 @@ public void Verify(string? workingDirectory, string? projectRootDirectory)
         var files = fileSystem.Directory.GetFiles(directoryPath);
         foreach (var fileName in this.SupportedConfigFileNames)
         {
-            this.log.Debug($"Trying to find configuration file {fileName} at '{directoryPath}'");
+            this.logger.LogDebug($"Trying to find configuration file {fileName} at '{directoryPath}'");
             var matchingFile = files.FirstOrDefault(file => string.Equals(FileSystemHelper.Path.GetFileName(file), fileName, StringComparison.OrdinalIgnoreCase));
             if (matchingFile == null) continue;
-            this.log.Info($"Found configuration file at '{matchingFile}'");
+            this.logger.LogInformation($"Found configuration file at '{matchingFile}'");
             return matchingFile;
         }
 

src/GitVersion.Configuration/ConfigurationProvider.cs

@@ -1,7 +1,7 @@
 using System.IO.Abstractions;
 using GitVersion.Configuration.Workflows;
 using GitVersion.Extensions;
-using GitVersion.Logging;
+using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
 using YamlDotNet.Core;
 
@@ -10,14 +10,14 @@ namespace GitVersion.Configuration;
 internal class ConfigurationProvider(
     IConfigurationFileLocator configFileLocator,
     IFileSystem fileSystem,
-    ILog log,
+    ILogger<ConfigurationProvider> logger,
     IConfigurationSerializer configurationSerializer,
     IOptions<GitVersionOptions> options)
     : IConfigurationProvider
 {
     private readonly IConfigurationFileLocator configFileLocator = configFileLocator.NotNull();
     private readonly IFileSystem fileSystem = fileSystem.NotNull();
-    private readonly ILog log = log.NotNull();
+    private readonly ILogger<ConfigurationProvider> logger = log.NotNull();
     private readonly IConfigurationSerializer configurationSerializer = configurationSerializer.NotNull();
     private readonly IOptions<GitVersionOptions> options = options.NotNull();
 
@@ -77,17 +77,17 @@ private IGitVersionConfiguration ProvideConfiguration(string? configFile,
     {
         if (configFilePath == null)
         {
-            this.log.Info("No configuration file found, using default configuration");
+            this.logger.LogInformation("No configuration file found, using default configuration");
             return null;
         }
 
         if (!this.fileSystem.File.Exists(configFilePath))
         {
-            this.log.Info($"Configuration file '{configFilePath}' not found");
+            this.logger.LogInformation($"Configuration file '{configFilePath}' not found");
             return null;
         }
 
-        this.log.Info($"Using configuration file '{configFilePath}'");
+        this.logger.LogInformation($"Using configuration file '{configFilePath}'");
         var content = fileSystem.File.ReadAllText(configFilePath);
         return configurationSerializer.Deserialize<Dictionary<object, object?>>(content);
     }

src/GitVersion.Core/Core/BranchesContainingCommitFinder.cs

@@ -1,13 +1,14 @@
 using GitVersion.Common;
 using GitVersion.Extensions;
 using GitVersion.Git;
+using Microsoft.Extensions.Logging;
 using GitVersion.Logging;
 
 namespace GitVersion;
 
-internal class BranchesContainingCommitFinder(IRepositoryStore repositoryStore, ILog log)
+internal class BranchesContainingCommitFinder(IRepositoryStore repositoryStore, ILogger<BranchesContainingCommitFinder> logger)
 {
-    private readonly ILog log = log.NotNull();
+    private readonly ILogger<BranchesContainingCommitFinder> logger = log.NotNull();
     private readonly IRepositoryStore repositoryStore = repositoryStore.NotNull();
 
     public IEnumerable<IBranch> GetBranchesContainingCommit(ICommit commit, IEnumerable<IBranch>? branches = null, bool onlyTrackedBranches = false)
@@ -23,16 +24,16 @@ public IEnumerable<IBranch> GetBranchesContainingCommit(ICommit commit, IEnumera
 
     private IEnumerable<IBranch> InnerGetBranchesContainingCommit(ICommit commit, IEnumerable<IBranch> branches, bool onlyTrackedBranches)
     {
-        using (log.IndentLog($"Getting branches containing the commit '{commit.Id}'."))
+        using (logger.IndentLog($"Getting branches containing the commit '{commit.Id}'."))
         {
             var directBranchHasBeenFound = false;
-            log.Info("Trying to find direct branches.");
+            logger.LogInformation("Trying to find direct branches.");
             // TODO: It looks wasteful looping through the branches twice. Can't these loops be merged somehow? @asbjornu
             var branchList = branches.ToList();
             foreach (var branch in branchList.Where(branch => BranchTipIsNullOrCommit(branch, commit) && !IncludeTrackedBranches(branch, onlyTrackedBranches)))
             {
                 directBranchHasBeenFound = true;
-                log.Info($"Direct branch found: '{branch}'.");
+                logger.LogInformation($"Direct branch found: '{branch}'.");
                 yield return branch;
             }
 
@@ -41,20 +42,20 @@ private IEnumerable<IBranch> InnerGetBranchesContainingCommit(ICommit commit, IE
                 yield break;
             }
 
-            log.Info($"No direct branches found, searching through {(onlyTrackedBranches ? "tracked" : "all")} branches.");
+            logger.LogInformation($"No direct branches found, searching through {(onlyTrackedBranches ? "tracked" : "all")} branches.");
             foreach (var branch in branchList.Where(b => IncludeTrackedBranches(b, onlyTrackedBranches)))
             {
-                log.Info($"Searching for commits reachable from '{branch}'.");
+                logger.LogInformation($"Searching for commits reachable from '{branch}'.");
 
                 var commits = this.repositoryStore.GetCommitsReacheableFrom(commit, branch);
 
                 if (!commits.Any())
                 {
-                    log.Info($"The branch '{branch}' has no matching commits.");
+                    logger.LogInformation($"The branch '{branch}' has no matching commits.");
                     continue;
                 }
 
-                log.Info($"The branch '{branch}' has a matching commit.");
+                logger.LogInformation($"The branch '{branch}' has a matching commit.");
                 yield return branch;
             }
         }