Skip to content

Commit db9f193

Browse files
authored
CI Updates (#85)
* fix mac cis * more pythons * remove stale builds * fix wheel uploads and only trigger on tags/workflow
1 parent 7cccd31 commit db9f193

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

.github/workflows/CI.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,13 @@ jobs:
2626
fail-fast: false
2727
matrix:
2828
os: [ubuntu-latest, macos-14, windows-latest]
29+
python-version: ["3.11", "3.12", "3.13"]
2930
steps:
3031
- uses: actions/checkout@v4
3132
- name: Run composite test suite
3233
uses: ./.github/actions/run-rust-python-tests
3334
with:
34-
python-version: "3.11"
35+
python-version: ${{ matrix.python-version }}
3536
rust-toolchain: stable
3637

3738
linux:
@@ -66,6 +67,7 @@ jobs:
6667
sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
6768
manylinux: auto
6869
- name: Upload wheels
70+
if: ${{ startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch' }}
6971
uses: actions/upload-artifact@v4
7072
with:
7173
name: wheels-linux-${{ matrix.platform.target }}
@@ -101,6 +103,7 @@ jobs:
101103
sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
102104
manylinux: musllinux_1_2
103105
- name: Upload wheels
106+
if: ${{ startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch' }}
104107
uses: actions/upload-artifact@v4
105108
with:
106109
name: wheels-musllinux-${{ matrix.platform.target }}
@@ -129,6 +132,7 @@ jobs:
129132
args: --release --out dist --find-interpreter
130133
sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
131134
- name: Upload wheels
135+
if: ${{ startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch' }}
132136
uses: actions/upload-artifact@v4
133137
with:
134138
name: wheels-windows-${{ matrix.platform.target }}
@@ -140,10 +144,10 @@ jobs:
140144
strategy:
141145
matrix:
142146
platform:
143-
- runner: macos-13
144-
target: x86_64
145147
- runner: macos-14
146148
target: aarch64
149+
- runner: macos-15
150+
target: aarch64
147151
steps:
148152
- uses: actions/checkout@v4
149153
- uses: actions/setup-python@v5
@@ -156,9 +160,10 @@ jobs:
156160
args: --release --out dist --find-interpreter
157161
sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
158162
- name: Upload wheels
163+
if: ${{ startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch' }}
159164
uses: actions/upload-artifact@v4
160165
with:
161-
name: wheels-macos-${{ matrix.platform.target }}
166+
name: wheels-macos-${{ matrix.platform.runner }}-${{ matrix.platform.target }}
162167
path: dist
163168

164169
sdist:
@@ -172,6 +177,7 @@ jobs:
172177
command: sdist
173178
args: --out dist
174179
- name: Upload sdist
180+
if: ${{ startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch' }}
175181
uses: actions/upload-artifact@v4
176182
with:
177183
name: wheels-sdist

test_python.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#!/usr/bin/env bash
22
set -e
33
source .venv/bin/activate
4-
maturin develop --release
4+
rm -f python/openai_harmony/openai_harmony.cpython-*.so
5+
maturin develop --release
56
pytest "$@"

0 commit comments

Comments
 (0)