From 296cf063a4a389dcbad7053355e58188c2c8bf48 Mon Sep 17 00:00:00 2001 From: Scott Beddall Date: Fri, 13 Sep 2024 15:15:13 -0700 Subject: [PATCH] eliminate the addition of buildDocs property. it requires powershell-yaml to be present on our base function. not good --- eng/common/scripts/Package-Properties.ps1 | 36 ----------------------- 1 file changed, 36 deletions(-) diff --git a/eng/common/scripts/Package-Properties.ps1 b/eng/common/scripts/Package-Properties.ps1 index aa4fa27e9d..58c5713cc6 100644 --- a/eng/common/scripts/Package-Properties.ps1 +++ b/eng/common/scripts/Package-Properties.ps1 @@ -15,7 +15,6 @@ class PackageProps [boolean]$IsNewSdk [string]$ArtifactName [string]$ReleaseStatus - [boolean]$BuildDocs # 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? @@ -82,41 +81,6 @@ class PackageProps $this.Initialize($name, $version, $directoryPath, $serviceDirectory) $this.Group = $group } - - hidden [void]InitializeBuildDocs( - [string]$ServiceDirectory - ) - { - # parse the relevant ci.yml file to determine if the package should build docs - $ciYmlPath = Join-Path $ServiceDirectory ".ci.yml" - - if (Test-Path $ciYmlPath) - { - $ciYml = ConvertFrom-Yaml (Get-Content $ciYmlPath -Raw) - - if ($ciYml.extends -and $ciYml.extends.parameters -and $ciYml.extends.parameters.Artifacts) { - $packagesBuildingDocs = $ciYml.extends.parameters.Artifacts ` - | Where-Object { - if ($_.PSObject.Properties["skipPublishDocsMs"]) { - $false - } - else { - $true - } - } ` - | Select-Object -ExpandProperty name - - if ($packagesBuildingDocs -contains $this.Name) - { - $this.BuildDocs = $true - } - } - } - else - { - $this.BuildDocs = $false - } - } } # Takes package name and service Name