Don't include Description as compiler metadata unconditionally

Since the description can contain newlines that will break the emitted editorconfig file with the values, only include in the specific scenario where we ensure the description isn't equal to the default.
devlooped · Mar 11, 2023 · 0dbce7c · 0dbce7c
1 parent fc6967c
commit 0dbce7c
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/CodeAnalysis/NuGetizer.CodeAnalysis.targets b/src/CodeAnalysis/NuGetizer.CodeAnalysis.targets
@@ -3,7 +3,6 @@
   <ItemGroup>
     <CompilerVisibleProperty Include="IsPacking" />
     <CompilerVisibleProperty Include="PackageId" />
-    <CompilerVisibleProperty Include="Description" />
     <CompilerVisibleProperty Include="PackageIcon" />
     <CompilerVisibleProperty Include="PackageIconUrl" />
     <CompilerVisibleProperty Include="PackageReadmeFile" />
@@ -22,4 +21,13 @@
   </ItemGroup>
   <Import Project="Devlooped.SponsorLink.targets" Condition="Exists('Devlooped.SponsorLink.targets')" />
 
+  <Target Name="_AddDescription" BeforeTargets="GenerateMSBuildEditorConfigFileShouldRun">
+    <ItemGroup>
+      <!-- We can't unconditionally emit Description since it can have newlines and that breaks editorconfig. -->
+      <CompilerVisibleProperty Include="Description" Condition="$(Description) == 'Package Description'" />
+    </ItemGroup>
+  </Target>
+
+  <Import Project="Devlooped.SponsorLink.targets" Condition="Exists('Devlooped.SponsorLink.targets')" />
+
 </Project>