Skip to content

Commit a99fc0a

Browse files
authored
Revert "Skip matrix jobs more cleanly (#183)"
This reverts commit dcbe514.
1 parent dcbe514 commit a99fc0a

File tree

1 file changed

+26
-18
lines changed

1 file changed

+26
-18
lines changed

.github/workflows/swift_package_test.yml

Lines changed: 26 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -218,13 +218,14 @@ on:
218218
jobs:
219219
macos-build:
220220
name: macOS (Xcode ${{ matrix.xcode_version }} - ${{ matrix.os_version }} - ${{ matrix.arch }})
221+
if: ${{ inputs.enable_macos_checks }}
221222
runs-on: [self-hosted, macos, "${{ matrix.os_version }}", "${{ matrix.arch }}"]
222223
strategy:
223224
fail-fast: false
224225
matrix:
225-
xcode_version: ${{ fromJson((inputs.enable_macos_checks && inputs.macos_xcode_versions) || '[]') }}
226-
os_version: ${{ fromJson((inputs.enable_macos_checks && inputs.macos_versions) || '[]') }}
227-
arch: ${{ fromJson((inputs.enable_macos_checks && inputs.macos_archs) || '[]') }}
226+
xcode_version: ${{ fromJson(inputs.macos_xcode_versions) }}
227+
os_version: ${{ fromJson(inputs.macos_versions) }}
228+
arch: ${{ fromJson(inputs.macos_archs) }}
228229
exclude:
229230
- ${{ fromJson(inputs.macos_exclude_xcode_versions) }}
230231
steps:
@@ -255,13 +256,14 @@ jobs:
255256

256257
ios-build:
257258
name: iOS (Build Only, Xcode ${{ matrix.xcode_version }} - ${{ matrix.os_version }} - ${{ matrix.arch }})
259+
if: ${{ inputs.enable_ios_checks }}
258260
runs-on: [self-hosted, macos, "${{ matrix.os_version }}", "${{ matrix.arch }}"]
259261
strategy:
260262
fail-fast: false
261263
matrix:
262-
xcode_version: ${{ fromJson((inputs.enable_ios_checks && (inputs.ios_host_xcode_versions || inputs.macos_xcode_versions)) || '[]') }}
263-
os_version: ${{ fromJson((inputs.enable_ios_checks && (inputs.ios_host_versions || inputs.macos_versions)) || '[]') }}
264-
arch: ${{ fromJson((inputs.enable_ios_checks && (inputs.ios_host_archs || inputs.macos_archs)) || '[]') }}
264+
xcode_version: ${{ fromJson(inputs.ios_host_xcode_versions || inputs.macos_xcode_versions) }}
265+
os_version: ${{ fromJson(inputs.ios_host_versions || inputs.macos_versions) }}
266+
arch: ${{ fromJson(inputs.ios_host_archs || inputs.macos_archs) }}
265267
exclude:
266268
- ${{ fromJson(inputs.ios_host_exclude_xcode_versions || inputs.macos_exclude_xcode_versions) }}
267269
steps:
@@ -290,12 +292,13 @@ jobs:
290292

291293
linux-build:
292294
name: Linux (${{ matrix.swift_version }} - ${{ matrix.os_version }})
295+
if: ${{ inputs.enable_linux_checks }}
293296
runs-on: ubuntu-latest
294297
strategy:
295298
fail-fast: false
296299
matrix:
297-
swift_version: ${{ fromJson((inputs.enable_linux_checks && inputs.linux_swift_versions) || '[]') }}
298-
os_version: ${{ fromJson((inputs.enable_linux_checks && inputs.linux_os_versions) || '[]') }}
300+
swift_version: ${{ fromJson(inputs.linux_swift_versions) }}
301+
os_version: ${{ fromJson(inputs.linux_os_versions) }}
299302
exclude:
300303
- ${{ fromJson(inputs.linux_exclude_swift_versions) }}
301304
container:
@@ -356,12 +359,13 @@ jobs:
356359

357360
linux-static-sdk-build:
358361
name: Static Linux Swift SDK Build (${{ matrix.swift_version }} - ${{ matrix.os_version }})
362+
if: ${{ inputs.enable_linux_static_sdk_build }}
359363
runs-on: ubuntu-latest
360364
strategy:
361365
fail-fast: false
362366
matrix:
363-
swift_version: ${{ fromJson((inputs.enable_linux_static_sdk_build && inputs.linux_static_sdk_versions) || '[]') }}
364-
os_version: ${{ fromJson((inputs.enable_linux_static_sdk_build && inputs.linux_os_versions) || '[]') }}
367+
swift_version: ${{ fromJson(inputs.linux_static_sdk_versions) }}
368+
os_version: ${{ fromJson(inputs.linux_os_versions) }}
365369
exclude:
366370
- ${{ fromJson(inputs.linux_static_sdk_exclude_swift_versions) }}
367371
container:
@@ -433,12 +437,13 @@ jobs:
433437
434438
wasm-sdk-build:
435439
name: Swift SDK for Wasm Build (${{ matrix.swift_version }} - ${{ matrix.os_version }})
440+
if: ${{ inputs.enable_wasm_sdk_build }}
436441
runs-on: ubuntu-latest
437442
strategy:
438443
fail-fast: false
439444
matrix:
440-
swift_version: ${{ fromJson((inputs.enable_wasm_sdk_build && inputs.wasm_sdk_versions) || '[]') }}
441-
os_version: ${{ fromJson((inputs.enable_wasm_sdk_build && inputs.linux_os_versions) || '[]') }}
445+
swift_version: ${{ fromJson(inputs.wasm_sdk_versions) }}
446+
os_version: ${{ fromJson(inputs.linux_os_versions) }}
442447
exclude:
443448
- ${{ fromJson(inputs.wasm_exclude_swift_versions) }}
444449
container:
@@ -510,12 +515,13 @@ jobs:
510515
511516
embedded-wasm-sdk-build:
512517
name: Embedded Swift SDK for Wasm Build (${{ matrix.swift_version }} - ${{ matrix.os_version }})
518+
if: ${{ inputs.enable_embedded_wasm_sdk_build }}
513519
runs-on: ubuntu-latest
514520
strategy:
515521
fail-fast: false
516522
matrix:
517-
swift_version: ${{ fromJson((inputs.enable_embedded_wasm_sdk_build && inputs.wasm_sdk_versions) || '[]') }}
518-
os_version: ${{ fromJson((inputs.enable_embedded_wasm_sdk_build && inputs.linux_os_versions) || '[]') }}
523+
swift_version: ${{ fromJson(inputs.wasm_sdk_versions) }}
524+
os_version: ${{ fromJson(inputs.linux_os_versions) }}
519525
exclude:
520526
- ${{ fromJson(inputs.wasm_exclude_swift_versions) }}
521527
container:
@@ -587,13 +593,14 @@ jobs:
587593
588594
android-sdk-build:
589595
name: Swift SDK for Android Build (${{ matrix.swift_version }} - ${{ matrix.os_version }} - NDK ${{ matrix.ndk_version }})
596+
if: ${{ inputs.enable_android_sdk_build }}
590597
runs-on: ubuntu-latest
591598
strategy:
592599
fail-fast: false
593600
matrix:
594-
swift_version: ${{ fromJson((inputs.enable_android_sdk_build && inputs.android_sdk_versions) || '[]') }}
595-
ndk_version: ${{ fromJson((inputs.enable_android_sdk_build && inputs.android_ndk_versions) || '[]') }}
596-
os_version: ${{ fromJson((inputs.enable_android_sdk_build && inputs.linux_os_versions) || '[]') }}
601+
swift_version: ${{ fromJson(inputs.android_sdk_versions) }}
602+
ndk_version: ${{ fromJson(inputs.android_ndk_versions) }}
603+
os_version: ${{ fromJson(inputs.linux_os_versions) }}
597604
exclude:
598605
- ${{ fromJson(inputs.android_exclude_swift_versions) }}
599606
container:
@@ -665,11 +672,12 @@ jobs:
665672
666673
windows-build:
667674
name: Windows (${{ matrix.swift_version }} - windows-2022)
675+
if: ${{ inputs.enable_windows_checks }}
668676
runs-on: windows-2022
669677
strategy:
670678
fail-fast: false
671679
matrix:
672-
swift_version: ${{ fromJson((inputs.enable_windows_checks && inputs.windows_swift_versions) || '[]') }}
680+
swift_version: ${{ fromJson(inputs.windows_swift_versions) }}
673681
exclude:
674682
- ${{ fromJson(inputs.windows_exclude_swift_versions) }}
675683
steps:

0 commit comments

Comments
 (0)