Sync eng/common directory with azure-sdk-tools for PR 7977 (#29115)

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#7977
Azure · Mar 29, 2024 · b5a1199 · b5a1199
1 parent 17923ab
commit b5a1199
Showing 1 changed file with 16 additions and 3 deletions.
diff --git a/eng/common/scripts/Create-APIReview.ps1 b/eng/common/scripts/Create-APIReview.ps1
@@ -153,6 +153,13 @@ function Submit-APIReview($packageInfo, $packagePath)
     }
 }
 
+function IsApiviewStatusCheckRequired($packageInfo)
+{
+    if (($packageInfo.SdkType -eq "client" -or $packageInfo.SdkType -eq "spring") -and $packageInfo.IsNewSdk) {
+        return $true
+    }
+    return $false
+}
 
 function ProcessPackage($packageName)
 {
@@ -234,8 +241,14 @@ function ProcessPackage($packageName)
                     # Check if package name is approved. Preview version cannot be released without package name approval
                     if (!$pkgNameStatus.IsApproved) 
                     {
-                        Write-Error $($pkgNameStatus.Details)
-                        return 1
+                        if (IsApiviewStatusCheckRequired $pkgInfo)
+                        {
+                            Write-Error $($pkgNameStatus.Details)
+                            return 1
+                        }
+                        else{
+                            Write-Host "Package name is not approved for package $($packageName), however it is not required for this package type so it can still be released without API review approval."
+                        }                        
                     }
                     # Ignore API review status for prerelease version
                     Write-Host "Package version is not GA. Ignoring API view approval status"
@@ -245,7 +258,7 @@ function ProcessPackage($packageName)
                     # Return error code if status code is 201 for new data plane package
                     # Temporarily enable API review for spring SDK types. Ideally this should be done be using 'IsReviewRequired' method in language side
                     # to override default check of SDK type client
-                    if (($pkgInfo.SdkType -eq "client" -or $pkgInfo.SdkType -eq "spring") -and $pkgInfo.IsNewSdk)
+                    if (IsApiviewStatusCheckRequired $pkgInfo)
                     {
                         if (!$apiStatus.IsApproved)
                         {