Fix failing tests - adjust to work with .NET 8.0 SDK environment and fix code formatting

Co-authored-by: pascalberger <2190718+pascalberger@users.noreply.github.com>

Author: Copilot

src/Cake.Issues.Tests/IssueReaderTests.cs

@@ -417,7 +417,6 @@ public void Should_Handle_Provider_Initialization_Failures_Concurrently()
             // Create a failing provider by passing null settings later
             var failingProvider = new FakeFailingIssueProvider(fixture.Log);
             fixture.IssueProviders.Add(failingProvider);
-            
             fixture.IssueProviders.Add(new FakeIssueProvider(fixture.Log, [
                 IssueBuilder.NewIssue("Success2", "ProviderType2", "ProviderName2")
                     .InFile(@"src\File2.cs", 2)
@@ -463,20 +462,19 @@ public void Should_Demonstrate_Parallel_Processing_Benefits_With_Simulated_Delay
 
             // Then
             issues.Count.ShouldBe(providerCount);
-            
+
             // With parallel processing, total time should be significantly less than 
             // sum of all delays (providerCount * delayPerProviderMs)
             var expectedSequentialTime = providerCount * delayPerProviderMs;
             var actualTime = stopwatch.ElapsedMilliseconds;
-            
+
             // Allow for some overhead but expect significant improvement
             var maxExpectedParallelTime = expectedSequentialTime * 0.4; // Should be much faster than 40% of sequential time
-            
+
             System.Console.WriteLine($"Sequential time would be ~{expectedSequentialTime}ms, parallel time was {actualTime}ms");
-            
+
             // This assertion may be flaky in CI environments, so we'll use a generous threshold
             actualTime.ShouldBeLessThan(expectedSequentialTime);
-            
             // Verify all issues have correct properties set
             // The Run property should be set to the same value from settings (even if null)
             issues.ShouldAllBe(x => x.Run == fixture.Settings.Run);