Skip to content

Commit e1eb2c5

Browse files
bryce-bbeekhc
authored andcommitted
Update workflow conditions for build and test jobs (open-telemetry#993)
1 parent 002f428 commit e1eb2c5

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

.github/workflows/BuildAndTest.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,10 @@ jobs:
2020
- name: Check if the workflow should run
2121
id: check
2222
run: |
23-
git diff --name-only origin/${{ github.base_ref }} HEAD | grep -q "Sources/" && echo "should-run=true" || echo "should-run=false"
23+
git diff --name-only origin/${{ github.base_ref }} HEAD | grep -q "Sources/" && echo "should-run=true" >> "$GITHUB_OUTPUT" || echo "should-run=false" >> "$GITHUB_OUTPUT"
2424
FormattingLint:
2525
needs: should-run
26-
if: ${{ needs.should-run.outputs.should-run }}
26+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
2727
runs-on: macos-15
2828
steps:
2929
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -32,7 +32,7 @@ jobs:
3232

3333
SwiftLint:
3434
needs: should-run
35-
if: ${{ needs.should-run.outputs.should-run }}
35+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
3636
runs-on: ubuntu-latest
3737
steps:
3838
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -43,7 +43,7 @@ jobs:
4343
DIFF_BASE: ${{ github.base_ref }}
4444
macOS:
4545
needs: should-run
46-
if: ${{ needs.should-run.outputs.should-run }}
46+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
4747
runs-on: macos-15
4848
steps:
4949
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -60,7 +60,7 @@ jobs:
6060
./codecov -f .build/debug/codecov/coverage_report.lcov
6161
iOS:
6262
needs: should-run
63-
if: ${{ needs.should-run.outputs.should-run }}
63+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
6464
runs-on: macos-15
6565
steps:
6666
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -75,7 +75,7 @@ jobs:
7575
run: make test-without-building-ios
7676
tvOS:
7777
needs: should-run
78-
if: ${{ needs.should-run.outputs.should-run }}
78+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
7979
runs-on: macos-15
8080
steps:
8181
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -90,7 +90,7 @@ jobs:
9090
run: make test-without-building-tvos
9191
watchOS:
9292
needs: should-run
93-
if: ${{ needs.should-run.outputs.should-run }}
93+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
9494
runs-on: macos-15
9595
steps:
9696
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -105,7 +105,7 @@ jobs:
105105
run: make test-without-building-watchos
106106
visionOS:
107107
needs: should-run
108-
if: ${{ needs.should-run.outputs.should-run }}
108+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
109109
runs-on: macos-15
110110
steps:
111111
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
@@ -120,7 +120,7 @@ jobs:
120120
run: make test-without-building-visionos
121121
linux:
122122
needs: should-run
123-
if: ${{ needs.should-run.outputs.should-run }}
123+
if: ${{ needs.should-run.outputs.should-run == 'true' }}
124124
runs-on: ubuntu-latest
125125
container: swift:6.2@sha256:0e4716bd34384d22963a63afbdbc93be3129dfd0753185aa1ded27755abdcae8
126126
steps:

0 commit comments

Comments
 (0)