chore: removed .NET Core 3.1 target framework from Correlate.AspNetCore (EOL) (#113)

Author: skwasjer

README.md

@@ -203,12 +203,12 @@ Please consider that .NET Core 3.1 and up now has built-in support for [W3C Trac
 
 ### Supported .NET targets
 - .NET 6.0, .NET 8.0
-- .NET Standard 2.0/.NET Core 3.1
+- .NET Standard 2.0
 
 ### ASP.NET Core support
 - ASP.NET Core 6.0/8.0
 
-### Build requirements
+### Build/test requirements
 - Visual Studio 2022
 - .NET 8 SDK
 - .NET 6 SDK

src/Correlate.AspNetCore/Correlate.AspNetCore.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>net8.0;net6.0;netcoreapp3.1TargetFrameworks>
+    <TargetFrameworks>net8.0;net6.0TargetFrameworks>
     <RootNamespace>CorrelateRootNamespace>
   PropertyGroup>
 

test/Correlate.AspNetCore.Tests/AspNetCore/IntegrationTests.cs

@@ -39,14 +39,6 @@ public IntegrationTests(TestAppFactory factory)
 
     public void Dispose()
     {
-#if NETCOREAPP3_1
-        // NET Core 3.1 test host does not trigger application stopping token, but our diagnostics observable
-        // (subscribed in UseCorrelate) must unsubscribe before next test run.
-        // This is a bit crude but later versions of .NET test host do it properly.
-        // ReSharper disable once ConditionalAccessQualifierIsNonNullableAccordingToAPIContract
-        IHostApplicationLifetime? appLifetime = _factory?.Services.GetRequiredService<IHostApplicationLifetime>();
-        appLifetime?.StopApplication();
-#endif
         // ReSharper disable ConditionalAccessQualifierIsNonNullableAccordingToAPIContract
         _factory?.Dispose();
         _mockHttp?.Dispose();

test/Correlate.AspNetCore.Tests/Correlate.AspNetCore.Tests.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Web">
 
   <PropertyGroup>
-    <TargetFrameworks>net8.0;net6.0;netcoreapp3.1TargetFrameworks>
+    <TargetFrameworks>net8.0;net6.0TargetFrameworks>
     <IsTestProject>trueIsTestProject>
     <RootNamespace>CorrelateRootNamespace>
   PropertyGroup>
@@ -16,11 +16,6 @@
     <Serilog_AspNetCore>6.1.0Serilog_AspNetCore>
   PropertyGroup>
 
-  <PropertyGroup Condition="'$(TargetFramework)'=='netcoreapp3.1'">
-    <Microsoft_AspNetCore_Mvc_Testing>3.1.32Microsoft_AspNetCore_Mvc_Testing>
-    <Serilog_AspNetCore>6.1.0Serilog_AspNetCore>
-  PropertyGroup>
-
   <ItemGroup>
     <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="$(Microsoft_AspNetCore_Mvc_Testing)" />
     <PackageReference Include="Serilog.AspNetCore" Version="$(Serilog_AspNetCore)" />