Undo <Nullable> enable change (moved to separate PR).

Author: tbdrake

Serilog.Sinks.Network.Test/JsonFormatter.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Diagnostics;
-using System.Linq;
 using System.Net;
 using System.Net.Sockets;
 using System.Threading.Tasks;
@@ -15,7 +14,10 @@ namespace Serilog.Sinks.Network.Test
 {
     public class JsonFormatter
     {
-        private static LoggerAndSocket ConfigureTestLogger(ITextFormatter? formatter = null, ILogEventEnricher[]? enrichers = null)
+        private static LoggerAndSocket ConfigureTestLogger(
+            ITextFormatter formatter = null,
+            ILogEventEnricher[] enrichers = null
+        )
         {
             var socket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);
             socket.Bind(new IPEndPoint(IPAddress.Loopback, 0));
@@ -67,7 +69,7 @@ public async Task IncludesCurrentActivityTraceAndSpanIds()
             // StartActivity() would return null if there were no listeners.
             using var activitySource = new ActivitySource("TestSource");
             using var activityListener = CreateAndAddActivityListener(activitySource.Name);
-            using Activity? activity = activitySource.StartActivity();
+            using var activity = activitySource.StartActivity();
             Assert.NotNull(activity);
 
             using var fixture = ConfigureTestLogger(new LogstashJsonFormatter());
@@ -98,7 +100,7 @@ public async Task DoesNotAddDuplicateTraceAndSpanIds()
         {
             using var activitySource = new ActivitySource("TestSource");
             using var activityListener = CreateAndAddActivityListener(activitySource.Name);
-            using Activity? activity = activitySource.StartActivity();
+            using var activity = activitySource.StartActivity();
             Assert.NotNull(activity);
 
             using var fixture = ConfigureTestLogger(

Serilog.Sinks.Network.Test/Serilog.Sinks.Network.Test.csproj

@@ -3,7 +3,6 @@
   <PropertyGroup>
     <TargetFramework>net6.0</TargetFramework>
     <LangVersion>Preview</LangVersion>
-    <Nullable>enable</Nullable>
   </PropertyGroup>
   <ItemGroup>
     <PackageReference Include="Serilog.Formatting.Compact" Version="3.0.0" />

Serilog.Sinks.Network.Test/WhenLoggingViaTcp.cs

@@ -15,7 +15,7 @@ namespace Serilog.Sinks.Network.Test
 {
     public class WhenLoggingViaTcp
     {
-        private static LoggerAndSocket ConfigureTestLogger(ITextFormatter? formatter = null)
+        private static LoggerAndSocket ConfigureTestLogger(ITextFormatter formatter = null)
         {
             var socket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);
             socket.Bind(new IPEndPoint(IPAddress.Loopback, 0));
@@ -66,7 +66,7 @@ public async Task CanLogWithProperties()
             using var fixture = ConfigureTestLogger();
             fixture.Logger.Information("TCP Hello {location}", "world");
             var receivedData = await ServerPoller.PollForReceivedData(fixture.Socket);
-            dynamic? payload = JsonConvert.DeserializeObject<ExpandoObject>(receivedData);
+            dynamic payload = JsonConvert.DeserializeObject<ExpandoObject>(receivedData);
             if (payload == null)
             {
                 throw new AssertionFailedException("expected payload not null");

Serilog.Sinks.Network.Test/WhenLoggingViaUdp.cs

@@ -15,7 +15,7 @@ namespace Serilog.Sinks.Network.Test
 {
     public class WhenLoggingViaUdp 
     {
-        private static LoggerAndSocket ConfigureTestLogger(ITextFormatter? formatter = null)
+        private static LoggerAndSocket ConfigureTestLogger(ITextFormatter formatter = null)
         {
             var socket = new Socket(AddressFamily.InterNetwork, SocketType.Dgram, ProtocolType.Udp);
             socket.Bind(new IPEndPoint(IPAddress.Loopback, 0));