Refactors RegisterTypes method to return IServiceCollection

Updates the `RegisterTypes` method signature across `IGitVersionModule` implementations and usage. This simplifies service registration by enabling method chaining in IoC configuration.

Author: arturcic

new-cli/GitVersion.Cli.Generator.Tests/SystemCommandlineGeneratorTests.cs

@@ -105,13 +105,14 @@ namespace {{Constants.GeneratedNamespaceName}};
 
 public class CommandsModule : IGitVersionModule
 {
-    public void RegisterTypes(IServiceCollection services)
+    public IServiceCollection RegisterTypes(IServiceCollection services)
     {
         services.AddSingleton<ICliApp, CliAppImpl>();
         services.AddSingleton<RootCommandImpl>();
 
         services.AddSingleton<TestCommand>();
         services.AddSingleton<ICommandImpl, TestCommandImpl>();
+        return services;
     }
 }
 """;

new-cli/GitVersion.Cli.Generator/SystemCommandlineContent.cs

@@ -109,7 +109,7 @@ namespace {{GeneratedNamespaceName}};
 
 public class CommandsModule : IGitVersionModule
 {
-    public void RegisterTypes(IServiceCollection services)
+    public IServiceCollection RegisterTypes(IServiceCollection services)
     {
         services.AddSingleton<ICliApp, CliAppImpl>();
         services.AddSingleton<RootCommandImpl>();
@@ -120,6 +120,7 @@ public void RegisterTypes(IServiceCollection services)
         services.AddSingleton<{{ if $command.CommandTypeNamespace != CommandNamespaceName }}{{$command.CommandTypeNamespace}}.{{ end }}{{$command.CommandTypeName}}>();
         services.AddSingleton<ICommandImpl, {{$command.CommandTypeName}}Impl>();
         {{~ end ~}}
+        return services;
     }
 }
 """;

new-cli/GitVersion.Cli/Program.cs

@@ -19,6 +19,7 @@
 
 var result = await app.RunAsync(args, cts.Token).ConfigureAwait(false);
 if (!Console.IsInputRedirected) Console.ReadKey();
+
 return result;
 
 static ServiceProvider RegisterModules(IEnumerable<IGitVersionModule> gitVersionModules)

new-cli/GitVersion.Common/Infrastructure/IGitVersionModule.cs

@@ -4,5 +4,5 @@ namespace GitVersion.Infrastructure;
 
 public interface IGitVersionModule
 {
-    void RegisterTypes(IServiceCollection services);
+    IServiceCollection RegisterTypes(IServiceCollection services);
 }

new-cli/GitVersion.Core.Libgit2Sharp/LibGit2SharpCoreModule.cs

@@ -5,5 +5,5 @@ namespace GitVersion.Git;
 
 public class LibGit2SharpCoreModule : IGitVersionModule
 {
-    public void RegisterTypes(IServiceCollection services) => services.AddSingleton<IGitRepository, GitRepository>();
+    public IServiceCollection RegisterTypes(IServiceCollection services) => services.AddSingleton<IGitRepository, GitRepository>();
 }

new-cli/GitVersion.Core/CoreModule.cs

@@ -6,10 +6,11 @@ namespace GitVersion;
 
 public class CoreModule : IGitVersionModule
 {
-    public void RegisterTypes(IServiceCollection services)
+    public IServiceCollection RegisterTypes(IServiceCollection services)
     {
         services.AddSingleton<IFileSystem, FileSystem>();
         services.AddSingleton<IEnvironment, Environment>();
         services.AddSingleton<IService, Service>();
+        return services;
     }
 }

new-cli/GitVersion.Core/Extensions/ServiceCollectionExtensions.cs

@@ -7,13 +7,7 @@ namespace GitVersion.Extensions;
 public static class ServiceCollectionExtensions
 {
     public static IServiceCollection RegisterModules(this IServiceCollection services, IEnumerable<IGitVersionModule> gitVersionModules)
-        => gitVersionModules.Aggregate(services, (current, gitVersionModule) => current.RegisterModule(gitVersionModule));
-
-    public static IServiceCollection RegisterModule(this IServiceCollection services, IGitVersionModule gitVersionModule)
-    {
-        gitVersionModule.RegisterTypes(services);
-        return services;
-    }
+        => gitVersionModules.Aggregate(services, (current, gitVersionModule) => gitVersionModule.RegisterTypes(current));
 
     public static IServiceCollection RegisterLogging(this IServiceCollection services)
     {