Use implicit environment variable instead of boolean coercion

Azure · Oct 2, 2024 · 342e7cf · 342e7cf
1 parent c877711
commit 342e7cf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/eng/pipelines/templates/steps/analyze.yml b/eng/pipelines/templates/steps/analyze.yml
@@ -23,4 +23,3 @@ steps:
       arguments: >
         -Toolchain '$(Toolchain)'
         -PackageInfoPath '$(Build.ArtifactStagingDirectory)/PackageInfo'
-        -SkipPackageAnalysis:('$(NoPackagesChanged)' -eq 'true')
diff --git a/eng/scripts/Analyze-Code.ps1 b/eng/scripts/Analyze-Code.ps1
@@ -30,7 +30,9 @@ Invoke-LoggedCommand "cargo +$Toolchain doc --workspace --no-deps"
 
 $verifyDependenciesScript = Join-Path $RepoRoot 'eng' 'scripts' 'verify-dependencies.rs' -Resolve
 
-if (!$SkipPackageAnalysis) {
+$skipPackages = $SkipPackageAnalysis -or $env:NoPackagesChanged
+
+if (!$skipPackages) {
   if (!(Test-Path $PackageInfoPath)) {
     Write-Error "Package info path '$PackageInfoPath' does not exist."
     exit 1