Skip to content

Commit 42936e3

Browse files
committed
put workflow_run success check in braces
1 parent 87a8522 commit 42936e3

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

.github/workflows/check-pkg-paths.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ on:
2323

2424
jobs:
2525
test:
26-
if: ${{ github.event.workflow_run.conclusion }} == 'success'
26+
if: ${{ github.event.workflow_run.conclusion == 'success'}}
2727
name: Test Package Paths
2828
runs-on: ubuntu-latest
2929

.github/workflows/health-metrics-pull-request.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ env:
3939
jobs:
4040
binary-size:
4141
name: Binary Size
42-
if: (${{ github.event.workflow_run.conclusion }} == 'success' && (github.event_name == 'push' || !(github.event.pull_request.head.repo.fork)) && (github.actor != 'dependabot[bot]'))
42+
if: (${{ github.event.workflow_run.conclusion == 'success'}} && (github.event_name == 'push' || !(github.event.pull_request.head.repo.fork)) && (github.actor != 'dependabot[bot]'))
4343
runs-on: ubuntu-latest
4444
steps:
4545
- uses: actions/checkout@v3
@@ -67,7 +67,7 @@ jobs:
6767
run: yarn size-report
6868
modular-export-size:
6969
name: Binary Size For Modular Exports
70-
if: (${{ github.event.workflow_run.conclusion }} == 'success' && (github.event_name == 'push' || !(github.event.pull_request.head.repo.fork)) && (github.actor != 'dependabot[bot]'))
70+
if: (${{ github.event.workflow_run.conclusion == 'success'}} && (github.event_name == 'push' || !(github.event.pull_request.head.repo.fork)) && (github.actor != 'dependabot[bot]'))
7171
runs-on: ubuntu-latest
7272
steps:
7373
- uses: actions/checkout@v3

.github/workflows/test-all.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
# Auth and Firestore are built and executed in their own jobs in an attempt to reduce flakiness.
3939
test-the-rest:
4040
name: (bulk) Node.js and Browser (Chrome) Tests
41-
if: ${{ github.event.workflow_run.conclusion }} == 'success'
41+
if: ${{ github.event.workflow_run.conclusion == 'success'}}
4242
runs-on: ubuntu-latest
4343
steps:
4444
# install Chrome first, so the correct version of webdriver can be installed by chromedriver when setting up the repo
@@ -137,7 +137,7 @@ jobs:
137137

138138
test-firestore:
139139
name: (Firestore) Node.js and Browser (Chrome) Tests
140-
if: ${{ github.event.workflow_run.conclusion }} == 'success'
140+
if: ${{ github.event.workflow_run.conclusion == 'success'}}
141141
runs-on: ubuntu-latest
142142
steps:
143143
# install Chrome so the correct version of webdriver can be installed by chromedriver when setting up the repo
@@ -180,7 +180,7 @@ jobs:
180180
path-to-lcov: ./lcov-all.info
181181
continue-on-error: true
182182
test-firestore-integration:
183-
if: ${{ github.event.workflow_run.conclusion }} == 'success'
183+
if: ${{ github.event.workflow_run.conclusion == 'success'}}
184184
strategy:
185185
fail-fast: false
186186
matrix:

0 commit comments

Comments
 (0)