Skip to content

Commit

Permalink
Sync eng/common directory with azure-sdk-tools for PR 8974 (Azure#41875)
Browse files Browse the repository at this point in the history
* 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 <scbedd@microsoft.com>
  • Loading branch information
azure-sdk and scbedd authored Sep 16, 2024
1 parent afe363c commit 932389b
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions eng/common/scripts/Package-Properties.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand All @@ -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"))
{
Expand Down Expand Up @@ -143,6 +147,7 @@ function Get-PrPkgProperties([string]$InputDiffJson) {
$key = $addition.Replace($RepoRoot, "").TrimStart('\/')

if ($lookup[$key]) {
$lookup[$key].IncludedForValidation = $true
$packagesWithChanges += $lookup[$key]
}
}
Expand Down

0 comments on commit 932389b

Please sign in to comment.