From 7f6a5a2ff642f93883be6df4bf363e1baf6f5283 Mon Sep 17 00:00:00 2001
From: Robert Dailey <git@rdailey.me>
Date: Tue, 28 Nov 2023 08:25:20 -0600
Subject: [PATCH] build: Remove global set of CLSCompliantAttribute

This was originally put in place in commit b0bbb23d3 (from 2021) to
silence analysis warnings. However, I observed no warnings so this must
have been corrected in dotnet since then.
---
 Directory.Build.props       | 7 -------
 tests/Directory.Build.props | 1 +
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/Directory.Build.props b/Directory.Build.props
index 333301e5..7236dde3 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -24,11 +24,4 @@
   <ItemGroup Condition="!$(ProjectName.EndsWith('.Tests'))">
     <InternalsVisibleTo Include="$(AssemblyName).Tests" />
   </ItemGroup>
-
-  <ItemGroup>
-    <!-- From https://github.com/dotnet/msbuild/pull/6285 -->
-    <AssemblyAttribute Include="System.CLSCompliantAttribute">
-      <_Parameter1>false</_Parameter1>
-    </AssemblyAttribute>
-  </ItemGroup>
 </Project>
diff --git a/tests/Directory.Build.props b/tests/Directory.Build.props
index aae9f18b..5afc2183 100644
--- a/tests/Directory.Build.props
+++ b/tests/Directory.Build.props
@@ -44,6 +44,7 @@
   </ItemGroup>
 
   <ItemGroup>
+    <!-- From https://github.com/dotnet/msbuild/pull/6285 -->
     <AssemblyAttribute Include="NUnit.Framework.ParallelizableAttribute">
       <_Parameter1>NUnit.Framework.ParallelScope.All</_Parameter1>
       <_Parameter1_IsLiteral>true</_Parameter1_IsLiteral>