From 0a3de43ccce9c85b5a993c051a45813896ee17aa Mon Sep 17 00:00:00 2001
From: Jordan Dominion <Cyberboss@users.noreply.github.com>
Date: Sat, 23 Dec 2023 11:54:30 -0500
Subject: [PATCH] Nullify `RepositoryManager`

---
 .../Components/Repository/RepositoryManager.cs     | 14 ++++++--------
 .../Extensions/FetchOptionsExtensions.cs           |  4 ++--
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/Tgstation.Server.Host/Components/Repository/RepositoryManager.cs b/src/Tgstation.Server.Host/Components/Repository/RepositoryManager.cs
index d733b1e506..727d171e4e 100644
--- a/src/Tgstation.Server.Host/Components/Repository/RepositoryManager.cs
+++ b/src/Tgstation.Server.Host/Components/Repository/RepositoryManager.cs
@@ -14,8 +14,6 @@ using Tgstation.Server.Host.IO;
 using Tgstation.Server.Host.Jobs;
 using Tgstation.Server.Host.Utils;
 
-#nullable disable
-
 namespace Tgstation.Server.Host.Components.Repository
 {
 	/// <inheritdoc />
@@ -120,12 +118,12 @@ namespace Tgstation.Server.Host.Components.Repository
 		}
 
 		/// <inheritdoc />
-		public async ValueTask<IRepository> CloneRepository(
+		public async ValueTask<IRepository?> CloneRepository(
 			Uri url,
-			string initialBranch,
-			string username,
-			string password,
-			JobProgressReporter progressReporter,
+			string? initialBranch,
+			string? username,
+			string? password,
+			JobProgressReporter? progressReporter,
 			bool recurseSubmodules,
 			CancellationToken cancellationToken)
 		{
@@ -210,7 +208,7 @@ namespace Tgstation.Server.Host.Components.Repository
 		}
 
 		/// <inheritdoc />
-		public async ValueTask<IRepository> LoadRepository(CancellationToken cancellationToken)
+		public async ValueTask<IRepository?> LoadRepository(CancellationToken cancellationToken)
 		{
 			logger.LogTrace("Begin LoadRepository...");
 			lock (semaphore)
diff --git a/src/Tgstation.Server.Host/Extensions/FetchOptionsExtensions.cs b/src/Tgstation.Server.Host/Extensions/FetchOptionsExtensions.cs
index e1b47d644a..6d71a68edf 100644
--- a/src/Tgstation.Server.Host/Extensions/FetchOptionsExtensions.cs
+++ b/src/Tgstation.Server.Host/Extensions/FetchOptionsExtensions.cs
@@ -27,7 +27,7 @@ namespace Tgstation.Server.Host.Extensions
 		public static FetchOptions Hydrate(
 			this FetchOptions fetchOptions,
 			ILogger logger,
-			JobProgressReporter progressReporter,
+			JobProgressReporter? progressReporter,
 			CredentialsHandler credentialsHandler,
 			CancellationToken cancellationToken)
 		{
@@ -63,7 +63,7 @@ namespace Tgstation.Server.Host.Extensions
 		/// <param name="progressReporter">The optional <see cref="JobProgressReporter"/> of the operation.</param>
 		/// <param name="cancellationToken">The <see cref="CancellationToken"/> for the operation.</param>
 		/// <returns>A new <see cref="LibGit2Sharp.Handlers.TransferProgressHandler"/> based on <paramref name="progressReporter"/>.</returns>
-		static TransferProgressHandler TransferProgressHandler(ILogger logger, JobProgressReporter progressReporter, CancellationToken cancellationToken) => transferProgress =>
+		static TransferProgressHandler TransferProgressHandler(ILogger logger, JobProgressReporter? progressReporter, CancellationToken cancellationToken) => transferProgress =>
 		{
 			double? percentage;
 			var totalObjectsToProcess = transferProgress.TotalObjects * 2;