Skip to content

Commit 6e61218

Browse files
committed
Make sure build-number gets tested in the done command as well; requires splitting the build-number action into build-number-report and build-number-done.
1 parent 850e2aa commit 6e61218

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Diff for: .github/workflows/test.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
strategy:
1515
matrix:
1616
os: [ubuntu-latest, windows-latest, macos-latest]
17-
action: [report, done, build-number] # Note: We're also testing 'install' since it's implicit in each action
17+
action: [report, done, build-number-report, build-number-done] # Note: We're also testing 'install' since it's implicit in each action
1818
fail_on_error: [true, false]
1919
steps:
2020
- name: Checkout code
@@ -42,8 +42,8 @@ jobs:
4242
github-token: ${{ secrets.GITHUB_TOKEN }}
4343
fail-on-error: ${{ matrix.fail_on_error }}
4444
debug: true
45-
build-number: ${{ matrix.action == 'build-number' && github.sha || '' }} # Only set 'build-number' to `${{ github.sha }}` when testing `build-number`
46-
parallel-finished: ${{ matrix.action == 'done' }} # Only set `parallel-finished` to `true` when testing `done`
45+
build-number: ${{ (matrix.action == 'build-number-report' || matrix.action == 'build-number-done') && github.sha || '' }} # Only set 'build-number' to `${{ github.sha }}` when testing `build-number-report` or `build-number-done`
46+
parallel-finished: ${{ matrix.action == 'done' || matrix.action == 'build-number-done' }} # Only set `parallel-finished` to `true` when testing `done` or `build-number-done`
4747
env:
4848
CI: true
4949
continue-on-error: ${{ matrix.fail_on_error }}

0 commit comments

Comments
 (0)