Merge pull request #740 from AzureAD/markzuber/strongnamer

Add PrivateAssets tag to StrongNamer in directory.build.props
AzureAD · Dec 19, 2018 · 3cfd9a1 · 3cfd9a1
2 parents 366c5c9 + e1bfbb4
commit 3cfd9a1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/Directory.Build.props b/Directory.Build.props
@@ -10,8 +10,12 @@
     <AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>
   </PropertyGroup>
   <ItemGroup>
+    <!-- PrivateAssets="All" is on these packages to ensure they're development dependencies
+         and aren't included as transitionary dependencies to package consumers.
+         For clarity, without PrivateAssets marked here, anyone consuming Microsoft.Identity.Client
+         would also be forced to install these dependencies.  PrivateAssets avoids this problem. -->
     <PackageReference Include="AsyncUsageAnalyzers" Version="1.0.0-alpha003" PrivateAssets="All" />
     <PackageReference Include="Microsoft.CodeAnalysis.FxCopAnalyzers" Version="2.6.2" PrivateAssets="All" />
-    <PackageReference Include="StrongNamer" Version="0.0.8"/>
+    <PackageReference Include="StrongNamer" Version="0.0.8" PrivateAssets="All" />
   </ItemGroup>
 </Project>