From a574487235e8c16c7779cd0eba6a113e80ed3d53 Mon Sep 17 00:00:00 2001
From: Jordan Dominion <Cyberboss@users.noreply.github.com>
Date: Sun, 17 Dec 2023 21:49:53 -0500
Subject: [PATCH] Nullify `IProvider`

---
 .../Components/Chat/Providers/IProvider.cs           | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/Tgstation.Server.Host/Components/Chat/Providers/IProvider.cs b/src/Tgstation.Server.Host/Components/Chat/Providers/IProvider.cs
index 0fc6beeb29..01b0d29a5e 100644
--- a/src/Tgstation.Server.Host/Components/Chat/Providers/IProvider.cs
+++ b/src/Tgstation.Server.Host/Components/Chat/Providers/IProvider.cs
@@ -7,8 +7,6 @@ using Tgstation.Server.Api.Models.Internal;
 using Tgstation.Server.Host.Components.Interop;
 using Tgstation.Server.Host.Models;
 
-#nullable disable
-
 namespace Tgstation.Server.Host.Components.Chat.Providers
 {
 	/// <summary>
@@ -47,7 +45,7 @@ namespace Tgstation.Server.Host.Components.Chat.Providers
 		/// <param name="cancellationToken">The <see cref="CancellationToken"/> for the operation.</param>
 		/// <returns>A <see cref="Task{TResult}"/> resulting in the next available <see cref="Message"/> or <see langword="null"/> if the <see cref="IProvider"/> needed to reconnect.</returns>
 		/// <remarks>Note that private messages will come in the form of <see cref="ChannelRepresentation"/>s not returned in <see cref="MapChannels(IEnumerable{ChatChannel}, CancellationToken)"/>.</remarks>
-		Task<Message> NextMessage(CancellationToken cancellationToken);
+		Task<Message?> NextMessage(CancellationToken cancellationToken);
 
 		/// <summary>
 		/// Gracefully disconnects the provider. Permanently stops the reconnection timer.
@@ -67,12 +65,12 @@ namespace Tgstation.Server.Host.Components.Chat.Providers
 		/// <summary>
 		/// Send a message to the <see cref="IProvider"/>.
 		/// </summary>
-		/// <param name="replyTo">The <see cref="Message"/> to reply to.</param>
+		/// <param name="replyTo">The optional <see cref="Message"/> to reply to.</param>
 		/// <param name="message">The <see cref="MessageContent"/>.</param>
 		/// <param name="channelId">The <see cref="ChannelRepresentation.RealId"/> to send to.</param>
 		/// <param name="cancellationToken">The <see cref="CancellationToken"/> for the operation.</param>
 		/// <returns>A <see cref="ValueTask"/> representing the running operation.</returns>
-		ValueTask SendMessage(Message replyTo, MessageContent message, ulong channelId, CancellationToken cancellationToken);
+		ValueTask SendMessage(Message? replyTo, MessageContent message, ulong channelId, CancellationToken cancellationToken);
 
 		/// <summary>
 		/// Set the interval at which the provider starts jobs to try to reconnect.
@@ -98,8 +96,8 @@ namespace Tgstation.Server.Host.Components.Chat.Providers
 			Models.RevisionInformation revisionInformation,
 			Api.Models.EngineVersion engineVersion,
 			DateTimeOffset? estimatedCompletionTime,
-			string gitHubOwner,
-			string gitHubRepo,
+			string? gitHubOwner,
+			string? gitHubRepo,
 			ulong channelId,
 			bool localCommitPushed,
 			CancellationToken cancellationToken);