refactor: Apply .NET 8 Languages

Author: guitarrapc

src/UnityBuildRunner.Core/SimpleConsoleLogger.cs

@@ -1,7 +1,6 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.Extensions.Logging;
-using System;
 
 namespace UnityBuildRunner.Core;
 
@@ -30,7 +29,7 @@ public SimpleConsoleLogger()
     {
     }
 
-    public IDisposable BeginScope<TState>(TState state)
+    public IDisposable? BeginScope<TState>(TState state) where TState : notnull
     {
         return NullDisposable.Instance;
     }
@@ -42,7 +41,7 @@ public bool IsEnabled(LogLevel logLevel)
 
     public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception? exception, Func<TState, Exception?, string> formatter)
     {
-        if (formatter == null) throw new ArgumentNullException(nameof(formatter));
+        ArgumentNullException.ThrowIfNull(formatter);
 
         var msg = formatter(state, exception);
 

src/UnityBuildRunner/Program.cs

@@ -1,41 +1,31 @@
 using Microsoft.Extensions.Logging;
-using System;
-using System.Linq;
-using System.Threading.Tasks;
 using UnityBuildRunner.Core;
 
 var builder = ConsoleApp.CreateBuilder(args);
 var app = builder.Build();
 app.AddCommands<UnityBuildRunnerCommand>();
 app.Run();
 
-public class UnityBuildRunnerCommand : ConsoleAppBase
+public class UnityBuildRunnerCommand(ILogger<UnityBuildRunnerCommand> logger) : ConsoleAppBase
 {
     private const string DefaultTimeout = "02:00:00"; // 2 hours
 
-    private readonly ILogger<UnityBuildRunnerCommand> logger;
-    private readonly TimeSpan timeoutDefault;
-
-    public UnityBuildRunnerCommand(ILogger<UnityBuildRunnerCommand> logger)
-    {
-        this.logger = logger;
-        timeoutDefault = TimeSpan.Parse(DefaultTimeout);
-    }
+    private readonly TimeSpan timeoutDefault = TimeSpan.Parse(DefaultTimeout);
 
     [RootCommand]
     public async Task<int> Run([Option("--unity-path", "Full Path to the Unity executable, leave empty when use 'UnityPath' Environment variables instead.")] string unityPath = "", [Option("--timeout", "Timeout for Unity Build.")] string timeout = DefaultTimeout)
     {
         var arguments = Context.Arguments
-            .Except(new[] { "--timeout", timeout });
+            .Except(["--timeout", timeout]);
         if (!string.IsNullOrEmpty(unityPath))
         {
-            arguments = arguments.Except(new[] { "--unity-path", unityPath });
+            arguments = arguments.Except(["--unity-path", unityPath]);
         }
         var args = arguments?.ToArray();
 
-        if (args is null || !args.Any())
+        if (args is null || args.Length == 0)
         {
-            throw new ArgumentOutOfRangeException($"No valid argument found, exiting. You have specified arguments: {string.Join(" ", args ?? Array.Empty<string>())}");
+            throw new ArgumentOutOfRangeException($"No valid argument found, exiting. You have specified arguments: {string.Join(" ", args ?? [])}");
         }
 
         // build

tests/UnityBuildRunner.Core.Tests/UnityBuildRunner.Core.Tests.csproj

@@ -5,12 +5,21 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="GitHubActionsTestLogger" />
-    <PackageReference Include="coverlet.msbuild" />
+    <PackageReference Include="GitHubActionsTestLogger">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+    </PackageReference>
+    <PackageReference Include="coverlet.msbuild">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+    </PackageReference>
     <PackageReference Include="FluentAssertions" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
     <PackageReference Include="xunit" />
-    <PackageReference Include="xunit.runner.visualstudio" />
+    <PackageReference Include="xunit.runner.visualstudio">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+    </PackageReference>
   </ItemGroup>
 
   <ItemGroup>