Skip to content

Commit 2efd339

Browse files
authored
[Infra] Fix missing Xcode 15 runs in functions and storage workflows (#14065)
1 parent 4e5fbe8 commit 2efd339

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

.github/workflows/functions.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,17 +30,17 @@ jobs:
3030
strategy:
3131
matrix:
3232
target: [ios, tvos, macos, watchos]
33-
include:
33+
build-env:
3434
- os: macos-14
3535
xcode: Xcode_15.2
3636
- os: macos-15
3737
xcode: Xcode_16.1
38-
runs-on: ${{ matrix.os }}
38+
runs-on: ${{ matrix.build-env.os }}
3939
steps:
4040
- uses: actions/checkout@v4
4141
- uses: ruby/setup-ruby@v1
4242
- name: Xcode
43-
run: sudo xcode-select -s /Applications/${{ matrix.xcode }}.app/Contents/Developer
43+
run: sudo xcode-select -s /Applications/${{ matrix.build-env.xcode }}.app/Contents/Developer
4444
- name: Setup Bundler
4545
run: scripts/setup_bundler.sh
4646
# The integration tests are flaky on Xcode 15 so only run the unit tests. The integration tests still run with SPM.

.github/workflows/storage.yml

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -195,15 +195,14 @@ jobs:
195195
strategy:
196196
matrix:
197197
target: [ios, tvos, macos, watchos]
198-
os: [macos-14]
199-
include:
198+
build-env:
200199
- os: macos-14
201200
xcode: Xcode_15.3
202201
tests: --skip-tests
203202
- os: macos-15
204203
xcode: Xcode_16.1
205204
tests: --test-specs=unit
206-
runs-on: ${{ matrix.os }}
205+
runs-on: ${{ matrix.build-env.os }}
207206
steps:
208207
- uses: actions/checkout@v4
209208
- uses: ruby/setup-ruby@v1
@@ -212,10 +211,10 @@ jobs:
212211
- name: Xcodes
213212
run: ls -l /Applications/Xcode*
214213
- name: Xcode
215-
run: sudo xcode-select -s /Applications/${{ matrix.xcode }}.app/Contents/Developer
214+
run: sudo xcode-select -s /Applications/${{ matrix.build-env.xcode }}.app/Contents/Developer
216215
- name: Build and test
217216
run: |
218-
scripts/third_party/travis/retry.sh scripts/pod_lib_lint.rb FirebaseStorage.podspec ${{ matrix.tests }} \
217+
scripts/third_party/travis/retry.sh scripts/pod_lib_lint.rb FirebaseStorage.podspec ${{ matrix.build-env.tests }} \
219218
--platforms=${{ matrix.target }}
220219
221220
storage-cron-only:
@@ -224,20 +223,19 @@ jobs:
224223
strategy:
225224
matrix:
226225
target: [ios, tvos, macos, watchos]
227-
os: [macos-14, macos-15]
228-
include:
226+
build-env:
229227
- os: macos-14
230228
xcode: Xcode_15.3
231229
- os: macos-15
232230
xcode: Xcode_16.1
233-
runs-on: ${{ matrix.os }}
231+
runs-on: ${{ matrix.build-env.os }}
234232
needs: pod-lib-lint
235233
steps:
236234
- uses: actions/checkout@v4
237235
- uses: ruby/setup-ruby@v1
238236
- name: Setup Bundler
239237
run: scripts/setup_bundler.sh
240238
- name: Xcode
241-
run: sudo xcode-select -s /Applications/${{ matrix.xcode }}.app/Contents/Developer
239+
run: sudo xcode-select -s /Applications/${{ matrix.build-env.xcode }}.app/Contents/Developer
242240
- name: PodLibLint Storage Cron
243241
run: scripts/third_party/travis/retry.sh scripts/pod_lib_lint.rb FirebaseStorage.podspec --platforms=${{ matrix.target }} --use-static-frameworks --skip-tests

0 commit comments

Comments
 (0)