[rel/3.8] Set packages README correctly (#5030)

Co-authored-by: Youssef1313 <youssefvictor00@gmail.com>

Author: youssef-backport-bot

Directory.Build.targets

@@ -11,4 +11,15 @@
     <AssemblyFileVersion>14.0.0.0</AssemblyFileVersion>
   </PropertyGroup>
 
+  <!-- Pack config -->
+  <PropertyGroup>
+    <PackageTags Condition=" '$(PackageTags)' == '' ">$(CommonPackageTags)</PackageTags>
+    <PackageReadmeFile Condition=" '$(PackageReadmeFile)' == '' and Exists('PACKAGE.md') ">PACKAGE.md</PackageReadmeFile>
+  </PropertyGroup>
+
+  <!-- Add a package README file from. -->
+  <ItemGroup Condition=" '$(PackageReadmeFile)' != '' ">
+    <None Include="$(PackageReadmeFile)" Pack="true" PackagePath="\" />
+  </ItemGroup>
+
 </Project>

src/Analyzers/MSTest.Analyzers.Package/MSTest.Analyzers.Package.csproj

@@ -10,7 +10,6 @@
   <PropertyGroup>
     <IsPackable>true</IsPackable>
     <PackageId>MSTest.Analyzers</PackageId>
-    <PackageReadmeFile>PACKAGE.md</PackageReadmeFile>
     <PackageTags>MSTest Analyzers TestFramework Unittest MSTestV2 Microsoft Test Testing TDD Framework</PackageTags>
     <PackageDescription>
       MSTest is Microsoft supported Test Framework.
@@ -29,7 +28,6 @@
 
   <ItemGroup>
     <None Update="tools\*.ps1" CopyToOutputDirectory="PreserveNewest" Pack="true" PackagePath="" />
-    <None Include="$(PackageReadmeFile)" Pack="true" PackagePath="\" />
   </ItemGroup>
 
   <Target Name="_GenerateGlobalConfigs">

src/Directory.Build.props

@@ -9,4 +9,8 @@
 
   <Import Project="../Directory.Build.props" />
 
+  <!-- Pack config -->
+  <PropertyGroup>
+    <CommonPackageTags>Microsoft test testing unittest unittesting unit-testing tdd</CommonPackageTags>
+  </PropertyGroup>
 </Project>

src/Package/MSTest/MSTest.csproj

@@ -20,7 +20,6 @@
     <IsPackable>true</IsPackable>
     <PackageId>MSTest</PackageId>
     <PackageTags>MSTest TestFramework TestAdapter VisualStudio Unittest MSTestV2 Microsoft</PackageTags>
-    <PackageReadmeFile>PACKAGE.md</PackageReadmeFile>
     <PackageDescription>
       MSTest is Microsoft supported Test Framework.
 
@@ -67,10 +66,6 @@
     <PackageReference Include="Microsoft.NET.Test.Sdk" VersionOverride="$(MicrosoftNETTestSdkVersion)" Condition="'$(TargetFramework)' != '$(UwpMinimum)'" PrivateAssets="none" />
   </ItemGroup>
 
-  <ItemGroup>
-    <None Include="$(PackageReadmeFile)" Pack="true" PackagePath="\" />
-  </ItemGroup>
-
   <!-- SDK bottom import -->
   <Import Project="Sdk.targets" Sdk="MSBuild.Sdk.Extras" Condition=" '$(OS)' == 'Windows_NT' " />
   <Import Project="Sdk.targets" Sdk="Microsoft.NET.Sdk" Condition=" '$(OS)' != 'Windows_NT' " />

src/Platform/Directory.Build.props

@@ -15,9 +15,4 @@
     <IsPackable>true</IsPackable>
   </PropertyGroup>
 
-  <!-- Pack config -->
-  <PropertyGroup>
-    <CommonPackageTags>Microsoft test testing unittest unittesting unit-testing tdd</CommonPackageTags>
-  </PropertyGroup>
-
 </Project>