Skip to content

Commit 61e7ce1

Browse files
committed
Try to improve the matrix and job names.
1 parent a7258b3 commit 61e7ce1

File tree

1 file changed

+10
-19
lines changed

1 file changed

+10
-19
lines changed

.github/workflows/build-and-test.yml

Lines changed: 10 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -169,24 +169,21 @@ jobs:
169169

170170
native:
171171
needs: determine-version
172-
name: "Native: ${{ matrix.os }}/${{ matrix.spm_configuration }}/${{ matrix.build_system }}/${{ matrix.cmake_generator }}"
172+
name: "Native: ${{ matrix.os }}/${{ matrix.spm_configuration }}/${{ matrix.build_system }}${{ matrix.build_system == 'cmake' && format(':{0}', matrix.cmake_generator) || '' }}"
173173
strategy:
174174
fail-fast: false
175175
matrix:
176176
os: [macos-14, ubuntu-latest, windows-latest]
177177
spm_configuration: [debug, release]
178-
cmake_generator: [Ninja, Xcode]
179178
build_system: [cmake, spm]
180179

181-
exclude:
182-
- os: ubuntu-latest
183-
cmake_generator: Xcode
184-
- os: windows-latest
180+
include:
181+
- cmake_generator: Ninja
182+
183+
- build_system: cmake
184+
os: macos-14
185185
cmake_generator: Xcode
186-
- build_system: spm
187-
cmake_generator: Ninja
188186

189-
include:
190187
- HYLO_LLVM_BUILD_RELEASE: 20250603-162600
191188
- HYLO_LLVM_BUILD_TYPE: MinSizeRel
192189
- HYLO_LLVM_DOWNLOAD_URL: https://github.com/hylo-lang/llvm-build/releases/download
@@ -204,9 +201,6 @@ jobs:
204201
triple_suffix: unknown-windows-msvc17
205202
windows_only: ''
206203

207-
- os: windows-latest
208-
use_spm: false
209-
210204
- os: macos-14
211205
triple_suffix: apple-darwin24.1.0
212206
triple_cpu: arm64
@@ -220,9 +214,6 @@ jobs:
220214
- spm_configuration: release
221215
cmake_build_type: Release
222216

223-
- cmake_generator: Xcode
224-
use_spm: false
225-
226217
runs-on: ${{ matrix.os }}
227218
env:
228219
llvm_url_prefix: ${{ matrix.HYLO_LLVM_DOWNLOAD_URL }}/${{ matrix.HYLO_LLVM_BUILD_RELEASE }}
@@ -297,7 +288,7 @@ jobs:
297288
if: ${{ matrix.build_system == 'cmake' }}
298289
run: ctest --output-on-failure --parallel --test-dir hylo/.cmake-build ${{ matrix.cmake_generator == 'Xcode' && format('-C {0}', matrix.cmake_build_type) || '' }}
299290

300-
- if: ${{ matrix.build_system == 'spm' && matrix.use_spm }}
291+
- if: ${{ matrix.build_system == 'spm' }}
301292
name: Create LLVM pkgconfig file and make it findable
302293
run: >-
303294
set -ex -o pipefail
@@ -310,7 +301,7 @@ jobs:
310301
echo 'PKG_CONFIG_PATH=${{ github.workspace }}/pkg-config' >> "$GITHUB_ENV"
311302
shell: bash
312303

313-
- if: ${{ matrix.build_system == 'spm' && matrix.use_spm }}
304+
- if: ${{ matrix.build_system == 'spm' }}
314305
# Workaround for https://github.com/actions/cache/issues/1541
315306
uses: actions/[email protected]
316307
name: SPM cache setup
@@ -336,15 +327,15 @@ jobs:
336327
}
337328
working-directory: hylo
338329

339-
- if: ${{ matrix.build_system == 'spm' && matrix.use_spm }}
330+
- if: ${{ matrix.build_system == 'spm' }}
340331
name: Build
341332
run: |
342333
swift build -c ${{ matrix.spm_configuration }} ${{ env.spm-build-options }} --build-tests
343334
${{ matrix.windows_only }} if (-not $?) { swift build ${{ env.spm-build-options }} --target BuildToolDependencies }
344335
345336
working-directory: hylo
346337

347-
- if: ${{ matrix.build_system == 'spm' && matrix.use_spm }}
338+
- if: ${{ matrix.build_system == 'spm' }}
348339
name: Test
349340
run: |
350341
swift test --skip-build -c ${{ matrix.spm_configuration }} ${{ env.spm-build-options }} ${{ env.spm-test-options }}

0 commit comments

Comments
 (0)