Skip to content

Commit

Permalink
Merge pull request natcap#1476 from davemfish/bugfix/GHA-1468-autotes…
Browse files Browse the repository at this point in the history
…t-fail

force autotest script to exit with code 1 if any models failed. Part 2
  • Loading branch information
dcdenu4 authored Dec 13, 2023
2 parents d43425c + 734695e commit 439fc80
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -338,13 +338,6 @@ jobs:
- name: Build binaries
run: make CONDA="$MAMBA_EXE" binaries

- name: Run invest-autotest with binaries
continue-on-error: true
if : |
github.event_name == 'push' &&
(contains(github.ref, 'release') || github.ref == 'refs/heads/main')
run: make invest_autotest

- name: Install Node.js
uses: actions/setup-node@v3
with:
Expand Down Expand Up @@ -460,6 +453,12 @@ jobs:
name: ${{ runner.os }}_puppeteer_log.zip'
path: ${{ matrix.puppeteer-log }}

- name: Run invest-autotest with binaries
if : |
github.event_name == 'push' &&
(startsWith(github.ref, 'release') || github.ref == 'refs/heads/main')
run: make invest_autotest

- name: Tar the workspace to preserve permissions (macOS)
if: failure() && matrix.os == 'macos-latest'
run: tar -cvf ${{ matrix.workspace-path}} ${{ github.workspace }}
Expand Down

0 comments on commit 439fc80

Please sign in to comment.