From da9c60ffd3b24213dab6b28d2379ba56f162e5b7 Mon Sep 17 00:00:00 2001 From: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com> Date: Mon, 16 Sep 2024 16:00:28 -0700 Subject: [PATCH] Sync eng/common directory with azure-sdk-tools for PR 8974 (#23445) * updating package properties with direct/indirect (if named differently) as well as pulling BuildDocs from ci.yml artifact list if it exists * eliminate the addition of buildDocs property. it requires powershell-yaml to be present on our base function. not good * remove call to InitializeBuildDocs --------- Co-authored-by: Scott Beddall --- eng/common/scripts/Package-Properties.ps1 | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/eng/common/scripts/Package-Properties.ps1 b/eng/common/scripts/Package-Properties.ps1 index e3cea7d392..fabe342388 100644 --- a/eng/common/scripts/Package-Properties.ps1 +++ b/eng/common/scripts/Package-Properties.ps1 @@ -15,6 +15,9 @@ class PackageProps [boolean]$IsNewSdk [string]$ArtifactName [string]$ReleaseStatus + # was this package purely included because other packages included it as an AdditionalValidationPackage? + [boolean]$IncludedForValidation + # does this package include other packages that we should trigger validation for? [string[]]$AdditionalValidationPackages PackageProps([string]$name, [string]$version, [string]$directoryPath, [string]$serviceDirectory) @@ -38,6 +41,7 @@ class PackageProps $this.Version = $version $this.DirectoryPath = $directoryPath $this.ServiceDirectory = $serviceDirectory + $this.IncludedForValidation = $false if (Test-Path (Join-Path $directoryPath "README.md")) { @@ -143,6 +147,7 @@ function Get-PrPkgProperties([string]$InputDiffJson) { $key = $addition.Replace($RepoRoot, "").TrimStart('\/') if ($lookup[$key]) { + $lookup[$key].IncludedForValidation = $true $packagesWithChanges += $lookup[$key] } }