Browse Source

ci: fixup doc-only-change on Windows (#35431)

John Kleinschmidt 2 years ago
parent
commit
6f77e63804
1 changed files with 7 additions and 7 deletions
  1. 7 7
      appveyor.yml

+ 7 - 7
appveyor.yml

@@ -64,11 +64,11 @@ for:
     build_script:
     - ps: |
         node script/yarn.js install --frozen-lockfile
-        $result = node script/doc-only-change.js --prNumber=$env:APPVEYOR_PULL_REQUEST_NUMBER --prBranch=$env:APPVEYOR_REPO_BRANCH
-        Write-Output $result
-        if ($result.ExitCode -eq 0) {
-          Write-warning "Skipping build for doc only change"; Exit-AppveyorBuild
+        node script/doc-only-change.js --prNumber=$env:APPVEYOR_PULL_REQUEST_NUMBER --prBranch=$env:APPVEYOR_REPO_BRANCH
+        if ($LASTEXITCODE -eq 0) {
+          Write-warning "Skipping tests for doc only change"; Exit-AppveyorBuild
         }
+        $global:LASTEXITCODE = 0
     - cd ..
     - ps: Write-Host "Building $env:GN_CONFIG build"
     - git config --global core.longpaths true
@@ -255,11 +255,11 @@ for:
     build_script:
     - ps: |
         node script/yarn.js install --frozen-lockfile
-        $result = node script/doc-only-change.js --prNumber=$env:APPVEYOR_PULL_REQUEST_NUMBER --prBranch=$env:APPVEYOR_REPO_BRANCH
-        Write-Output $result
-        if ($result.ExitCode -eq 0) {
+        node script/doc-only-change.js --prNumber=$env:APPVEYOR_PULL_REQUEST_NUMBER --prBranch=$env:APPVEYOR_REPO_BRANCH
+        if ($LASTEXITCODE -eq 0) {
           Write-warning "Skipping tests for doc only change"; Exit-AppveyorBuild
         }
+        $global:LASTEXITCODE = 0
     - ps: |
         cd ..
         mkdir out\Default