Skip to content

Commit

Permalink
eliminate the addition of buildDocs property. it requires powershell-…
Browse files Browse the repository at this point in the history
…yaml to be present on our base function. not good
  • Loading branch information
scbedd authored and azure-sdk committed Sep 13, 2024
1 parent e320abf commit 296cf06
Showing 1 changed file with 0 additions and 36 deletions.
36 changes: 0 additions & 36 deletions eng/common/scripts/Package-Properties.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -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?
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 296cf06

Please sign in to comment.