Skip to content

Commit c6df49e

Browse files
CopilotMalcolmnixon
andcommitted
Clarify PackageReference entries with comments and fix Polyfill formatting consistency
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
1 parent 571572f commit c6df49e

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

src/DemaConsulting.TemplateDotNetTool/DemaConsulting.TemplateDotNetTool.csproj

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,15 @@
4848

4949
<ItemGroup>
5050
<PackageReference Include="DemaConsulting.TestResults" Version="1.5.0" />
51+
<!-- PrivateAssets="All" prevents these build-time-only packages from becoming transitive dependencies -->
5152
<PackageReference Include="Microsoft.Sbom.Targets" Version="4.1.5" PrivateAssets="All" />
5253
<PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.103" PrivateAssets="All" />
54+
<!--
55+
Analyzer and source-only packages require both PrivateAssets and IncludeAssets:
56+
PrivateAssets="all" - prevents these build-time assets from flowing to consumers
57+
IncludeAssets - explicitly enables contentfiles (source injection) and
58+
analyzers/buildtransitive (Roslyn analyzers at compile time)
59+
-->
5360
<PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="10.0.103">
5461
<PrivateAssets>all</PrivateAssets>
5562
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
@@ -58,7 +65,9 @@
5865
<PrivateAssets>all</PrivateAssets>
5966
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
6067
</PackageReference>
61-
<PackageReference Include="Polyfill" Version="9.12.0" PrivateAssets="All">
68+
<!-- Polyfill is a source-only package; contentfiles delivers the polyfill source into this project -->
69+
<PackageReference Include="Polyfill" Version="9.12.0">
70+
<PrivateAssets>all</PrivateAssets>
6271
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
6372
</PackageReference>
6473
</ItemGroup>

0 commit comments

Comments
 (0)