Add AOT analyzers

Tyrrrz · May 20, 2024 · 2a2fe67 · 2a2fe67
1 parent eaac7e9
commit 2a2fe67
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/CliWrap/CliWrap.csproj b/CliWrap/CliWrap.csproj
@@ -1,8 +1,10 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFrameworks>netstandard2.0;netstandard2.1;netcoreapp3.0;net462</TargetFrameworks>
+    <TargetFrameworks>netstandard2.0;netstandard2.1;netcoreapp3.0;net462;net80</TargetFrameworks>
     <IsPackable>true</IsPackable>
+    <IsTrimmable Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net6.0'))">true</IsTrimmable>
+    <IsAotCompatible Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net7.0'))">true</IsAotCompatible>
   </PropertyGroup>
 
   <PropertyGroup>

diff --git a/CliWrap/Command.Execution.cs b/CliWrap/Command.Execution.cs
@@ -92,6 +92,8 @@ private ProcessStartInfo CreateStartInfo()
         // Set credentials
         try
         {
+            // Disable CA1416 because we're handling an exception that is thrown by the property setters
+#pragma warning disable CA1416
             if (Credentials.Domain is not null)
                 startInfo.Domain = Credentials.Domain;
 
@@ -103,6 +105,7 @@ private ProcessStartInfo CreateStartInfo()
 
             if (Credentials.LoadUserProfile)
                 startInfo.LoadUserProfile = Credentials.LoadUserProfile;
+#pragma warning restore CA1416
         }
         catch (NotSupportedException ex)
         {