Skip to content

Commit d7eb494

Browse files
committed
Use include instead of build in matrix (#37)
The name `include` is more commonly used for the matrix definition and doing so makes it possible to reference fields as, e.g., `matrix.os` instead of `matrix.build.os`.
1 parent 5f5ad14 commit d7eb494

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,11 @@ concurrency:
4141

4242
jobs:
4343
build_dist:
44-
runs-on: ${{matrix.build.os}}
44+
runs-on: ${{matrix.os}}
4545
strategy:
4646
fail-fast: false
4747
matrix:
48-
build: [
48+
include: [
4949
{os: ubuntu-latest, dist: cp310-manylinux_x86_64, arch: x86_64},
5050
{os: ubuntu-latest, dist: cp311-manylinux_x86_64, arch: x86_64},
5151
{os: ubuntu-latest, dist: cp312-manylinux_x86_64, arch: x86_64},
@@ -89,9 +89,9 @@ jobs:
8989
{os: windows-latest, dist: cp314-win_amd64, arch: AMD64},
9090
]
9191
env:
92-
MACOSX_DEPLOYMENT_TARGET: ${{matrix.build.target}}
93-
CIBW_BUILD: ${{matrix.build.dist}}
94-
CIBW_ARCHS: ${{matrix.build.arch}}
92+
MACOSX_DEPLOYMENT_TARGET: ${{matrix.target}}
93+
CIBW_BUILD: ${{matrix.dist}}
94+
CIBW_ARCHS: ${{matrix.arch}}
9595
CIBW_TEST_REQUIRES: pytest pytest-xdist stim~=1.15 sinter pygltflib
9696
CIBW_TEST_COMMAND: pytest -n auto {project}/src
9797
CIBW_PRERELEASE_PYTHONS: 1
@@ -106,9 +106,9 @@ jobs:
106106
- run: python tools/overwrite_dev_versions_with_date.py
107107
- run: python -m pip install pybind11~=2.11.1 cibuildwheel~=3.2.0 setuptools
108108
- run: python -m cibuildwheel --print-build-identifiers
109-
- if: matrix.build.arch != 'AMD64'
109+
- if: matrix.arch != 'AMD64'
110110
run: python -m cibuildwheel --output-dir dist
111-
- if: matrix.build.arch == 'AMD64'
111+
- if: matrix.arch == 'AMD64'
112112
env:
113113
# These paths are on GitHub runners.
114114
CIBW_REPAIR_WHEEL_COMMAND_WINDOWS: 'delvewheel repair -v --add-path "D:\\a\\chromobius\\chromobius\\dist;D:\\a\\chromobius\\chromobius" -w {dest_dir} {wheel}'
@@ -119,7 +119,7 @@ jobs:
119119
python -m cibuildwheel --output-dir dist
120120
- uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
121121
with:
122-
name: "dist-chromobius-${{matrix.build.os}}-${{matrix.build.dist}}-${{matrix.build.arch}}"
122+
name: "dist-chromobius-${{matrix.os}}-${{matrix.dist}}-${{matrix.arch}}"
123123
path: dist/*
124124
build_sdist:
125125
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)