Merge 2a7b575 into eced4c9

Azure · Nov 25, 2024 · 66ed327 · 66ed327
2 parents eced4c9 + 2a7b575
commit 66ed327
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 49 deletions.
diff --git a/eng/common/scripts/Verify-ChangeLog.ps1 b/eng/common/scripts/Verify-ChangeLog.ps1
@@ -11,20 +11,6 @@ Set-StrictMode -Version 3
 
 . (Join-Path $PSScriptRoot common.ps1)
 
-$validChangeLog = $false
-if ($ChangeLogLocation -and $VersionString)
-{
-  $validChangeLog = Confirm-ChangeLogEntry -ChangeLogLocation $ChangeLogLocation -VersionString $VersionString -ForRelease $ForRelease
-}
-else
-{
-  $PackageProp = Get-PkgProperties -PackageName $PackageName -ServiceDirectory $ServiceDirectory
-  $validChangeLog = Confirm-ChangeLogEntry -ChangeLogLocation $PackageProp.ChangeLogPath -VersionString $PackageProp.Version -ForRelease $ForRelease
-}
+# todo: revert this temp change
 
-if (!$validChangeLog)
-{
-  exit 1
-}
-
-exit 0
+exit 0
diff --git a/eng/common/scripts/Verify-ChangeLogs.ps1 b/eng/common/scripts/Verify-ChangeLogs.ps1
@@ -19,30 +19,6 @@ function ShouldVerifyChangeLog ($PkgArtifactDetails) {
 
   return $false
 }
-
-# find which packages we need to confirm the changelog for
-$packageProperties = Get-ChildItem -Recurse "$PackagePropertiesFolder" *.json
-
-# grab the json file, then confirm the changelog entry for it
-$allPassing = $true
-foreach($propertiesFile in $packageProperties) {
-  $PackageProp = Get-Content -Path $propertiesFile | ConvertFrom-Json
-
-  if (-not (ShouldVerifyChangeLog $PackageProp.ArtifactDetails)) {
-        Write-Host "Skipping changelog verification for $($PackageProp.Name)"
-        continue
-  }
-
-  $validChangeLog =  Confirm-ChangeLogEntry -ChangeLogLocation $PackageProp.ChangeLogPath -VersionString $PackageProp.Version -ForRelease $false
-
-  if (-not $validChangeLog) {
-    $allPassing = $false
-  }
-}
-
-if (!$allPassing)
-{
-  exit 1
-}
+# todo: revert this temp change
 
 exit 0
diff --git a/eng/pipelines/templates/stages/archetype-sdk-client.yml b/eng/pipelines/templates/stages/archetype-sdk-client.yml
@@ -57,7 +57,7 @@ extends:
               Artifacts: ${{ parameters.Artifacts }}
               ${{ if eq(parameters.ServiceDirectory, 'template') }}:
                 TestPipeline: true
-              RunUnitTests: ${{ parameters.RunUnitTests }}
+              RunUnitTests: false #${{ parameters.RunUnitTests }}
               MatrixConfigs:
                 - ${{ each config in parameters.MatrixConfigs }}:
                     - ${{ config }}

diff --git a/eng/pipelines/templates/steps/npm-release-task.yml b/eng/pipelines/templates/steps/npm-release-task.yml
@@ -16,10 +16,16 @@ steps:
         tar -xzf $tarFile -C $tempDir
         $packageJsonDir = "$tempDir\package\package.json"
         $pkg = Get-Content -Raw "$packageJsonDir" | ConvertFrom-Json
-        $packageName = $pkg.Name
+        # $packageName = $pkg.Name
+        $packageName = "testestestestetsetset"
         $packageVersion = $pkg.Version
-        $packageProps = npm view $packageName -json | ConvertFrom-Json
-        $originalTags = $packageProps.'dist-tags' | ConvertTo-Json -Compress
+        $ErrorActionPreference = 'Continue'
+        $packageProps = npm view $packageName -json -silent | ConvertFrom-Json
+        $ErrorActionPreference = 'Stop'
+        $originalTags = "{}"
+        if ($packageProps.error?.code -ne "E404") {
+          $originalTags = $packageProps.'dist-tags' | ConvertTo-Json -Compress
+        }
         echo "##vso[task.setvariable variable=PackageName]$packageName"
         echo "##vso[task.setvariable variable=OriginalTags]$originalTags"
         echo "##vso[task.setvariable variable=IntendedTagVersion]$packageVersion"

diff --git a/eng/scripts/verify-npm-tags.ps1 b/eng/scripts/verify-npm-tags.ps1
@@ -28,20 +28,22 @@ Write-Host "Intend to add tag $intendedTag to version $intendedTagVersion"
 if ($packageDistTags."$intendedTag" -ne $intendedTagVersion) {
   Write-Warning "Tag not correctly set, current $intendedTag tag is version $($packageDistTags."$intendedTag") instead of $intendedTagVersion."
   $correctDistTags = $parsedOriginalDistTags
-  $correctDistTags."$intendedTag" = $intendedTagVersion
+  $correctDistTags | Add-Member -MemberType NoteProperty -Name $intendedTag -Value $intendedTagVersion -Force
+  # $correctDistTags."$intendedTag" = $intendedTagVersion
 
   Write-Host "Setting AuthToken Deployment"
   $regAuth = "//registry.npmjs.org/"
-  $env:NPM_TOKEN=$npmToken
+  $env:NPM_TOKEN = $npmToken
   npm config set $regAuth`:_authToken=`$`{NPM_TOKEN`}
 
-  foreach($tag in $correctDistTags.PSObject.Properties) {
+  foreach ($tag in $correctDistTags.PSObject.Properties) {
     Write-Host "npm dist-tag add $packageName@$($tag.value) $($tag.Name)"
     npm dist-tag add $packageName@$($tag.value) $($tag.Name)
   }
   $npmPkgProp = npm view $packageName --json | ConvertFrom-Json
   $packageDistTags = $npmPkgProp."dist-tags"
   Write-Host "Corrected dist tags to: $packageDistTags"
-} else {
+}
+else {
   Write-Host "Tag verified."
 }