Skip to content

Commit

Permalink
Docs onboarding: ValidatePackageForOnboarding2 -> PackageIsValidForDo…
Browse files Browse the repository at this point in the history
…csOnboarding (#2930)

Co-authored-by: Daniel Jurek <djurek@microsoft.com>
  • Loading branch information
azure-sdk and danieljurek authored Sep 26, 2024
1 parent 5287714 commit 4a78ff5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions eng/common/scripts/Update-DocsMsPackages.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ function GetMetadata($moniker) {
return $metadata
}

function ValidatePackageForOnboarding2($package) {
function PackageIsValidForDocsOnboarding($package) {
if (!(Test-Path "Function:$ValidateDocsMsPackagesFn")) {
return $true
}
Expand Down Expand Up @@ -88,7 +88,7 @@ foreach ($moniker in $MONIKERS) {
if ($package.ContainsKey('_SkipDocsValidation') -and $true -eq $package['_SkipDocsValidation']) {
Write-Host "Skip validation for package: $($packageIdentity)"
}
elseif (!(ValidatePackageForOnboarding2 $package)) {
elseif (!(PackageIsValidForDocsOnboarding $package)) {
LogWarning "Skip adding package that did not pass validation: $($packageIdentity)"
continue
}
Expand All @@ -101,7 +101,7 @@ foreach ($moniker in $MONIKERS) {
$oldPackage = $alreadyOnboardedPackages[$packageIdentity]

if ($oldPackage.Version -ne $package.Version) {
if (!(ValidatePackageForOnboarding2 $package)) {
if (!(PackageIsValidForDocsOnboarding $package)) {
LogWarning "Omitting package that failed validation: $($packageIdentity)@$($package.Version)"
continue
}
Expand Down

0 comments on commit 4a78ff5

Please sign in to comment.