Sync eng/common directory with azure-sdk-tools for PR 9635 (Azure#47850)

* ensure that packages that come back from AdditionalValidationPackages are NOT counted as 'includedForValidation: true' if they also exist in the original 'these packages changed' set Co-authored-by: Ben Broderick Phillips <[email protected]> --------- Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]>
azure-sdk · Jan 15, 2025 · e2f80fa · e2f80fa
1 parent b8e74aa
commit e2f80fa
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/eng/common/scripts/Package-Properties.ps1 b/eng/common/scripts/Package-Properties.ps1
@@ -187,12 +187,17 @@ function Get-PrPkgProperties([string]$InputDiffJson) {
         }
     }
 
+    $existingPackageNames = @($packagesWithChanges | ForEach-Object { $_.Name })
     foreach ($addition in $additionalValidationPackages) {
         $key = $addition.Replace($RepoRoot, "").TrimStart('\/')
 
         if ($lookup[$key]) {
-            $lookup[$key].IncludedForValidation = $true
-            $packagesWithChanges += $lookup[$key]
+            $pkg = $lookup[$key]
+
+            if ($pkg.Name -notin $existingPackageNames) {
+                $pkg.IncludedForValidation = $true
+                $packagesWithChanges += $pkg
+            }
         }
     }