Skip to content

Commit 70c096c

Browse files
Yvictorclaude
andcommitted
fix: revert Linux x86_64 to original working configuration
- Revert Linux x86_64 back to Ta-Lib v0.4.0 (was working originally) - Remove extra environment variables that were causing test failures - Simplify artifact naming back to 'wheels' to match original structure - Remove duplicate macOS jobs and consolidate back to single job 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent fba05c3 commit 70c096c

File tree

1 file changed

+15
-54
lines changed

1 file changed

+15
-54
lines changed

.github/workflows/CI.yml

Lines changed: 15 additions & 54 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
runs-on: ubuntu-latest
2727
strategy:
2828
matrix:
29-
target: [x86_64, aarch64-unknown-linux-gnu]
29+
target: [x86_64]
3030
steps:
3131
- uses: actions/checkout@v4
3232
- uses: actions/setup-python@v5
@@ -36,14 +36,14 @@ jobs:
3636
uses: astral-sh/setup-uv@v5
3737
- name: Install talib
3838
run: |
39-
wget https://github.com/ta-lib/ta-lib/releases/download/v0.6.4/ta-lib-0.6.4-src.tar.gz
40-
tar -xzf ta-lib-0.6.4-src.tar.gz
41-
cd ta-lib-0.6.4
39+
wget https://github.com/Yvictor/polars_ta_extension/releases/download/0.1.0/ta-lib-0.4.0-src.tar.gz
40+
tar -xzf ta-lib-0.4.0-src.tar.gz
41+
cd ta-lib
4242
./configure
4343
make
4444
sudo make install
4545
cd ..
46-
rm -rf ta-lib-0.6.4
46+
rm -rf ta-lib
4747
env:
4848
TA_INCLUDE_PATH: ${{ github.workspace }}/dependencies/include
4949
TA_LIBRARY_PATH: ${{ github.workspace }}/dependencies/lib
@@ -52,13 +52,11 @@ jobs:
5252
uv run pytest
5353
env:
5454
DEPS_PATH: ${{ github.workspace }}/dependencies
55-
TA_INCLUDE_PATH: /usr/local/include:${{ github.workspace }}/dependencies/include
56-
TA_LIBRARY_PATH: /usr/local/lib:${{ github.workspace }}/dependencies/lib
57-
LD_LIBRARY_PATH: /usr/local/lib:$LD_LIBRARY_PATH
58-
PKG_CONFIG_PATH: /usr/local/lib/pkgconfig:$PKG_CONFIG_PATH
55+
TA_INCLUDE_PATH: ${{ github.workspace }}/dependencies/include
56+
TA_LIBRARY_PATH: ${{ github.workspace }}/dependencies/lib
5957
- name: Build wheels for x86_64
6058
if: matrix.target == 'x86_64'
61-
uses: PyO3/maturin-action@v1
59+
uses: PyO3/maturin-action@v1.46.0
6260
with:
6361
target: ${{ matrix.target }}
6462
args: --release --out dist --find-interpreter
@@ -113,7 +111,7 @@ jobs:
113111
- name: Upload wheels
114112
uses: actions/upload-artifact@v4
115113
with:
116-
name: wheels-linux-${{ matrix.target }}
114+
name: wheels
117115
path: dist
118116
- name: Test wheel installation
119117
run: |
@@ -159,7 +157,7 @@ jobs:
159157
- name: Upload wheels
160158
uses: actions/upload-artifact@v4
161159
with:
162-
name: wheels-linux-${{ matrix.target }}
160+
name: wheels
163161
path: dist
164162
- name: Test wheel installation
165163
run: |
@@ -204,54 +202,19 @@ jobs:
204202
- name: Upload wheels
205203
uses: actions/upload-artifact@v4
206204
with:
207-
name: wheels-windows-${{ matrix.target }}
205+
name: wheels
208206
path: dist
209207
- name: Test wheel installation
210208
run: |
211209
pip install dist/*.whl
212210
python -c "import polars_talib; print('Import successful')"
213211
shell: bash
214212

215-
macos-intel:
216-
runs-on: macos-13
217-
strategy:
218-
matrix:
219-
target: [x86_64]
220-
steps:
221-
- uses: actions/checkout@v4
222-
- uses: actions/setup-python@v5
223-
with:
224-
python-version: '3.10'
225-
- name: install uv
226-
uses: astral-sh/setup-uv@v5
227-
- name: install tools
228-
run: |
229-
uv tool install maturin
230-
- name: Build wheels
231-
uses: PyO3/maturin-action@v1
232-
with:
233-
target: ${{ matrix.target }}
234-
args: --release --out dist --find-interpreter
235-
sccache: 'true'
236-
env:
237-
DEPS_PATH: ${{ github.workspace }}/dependencies
238-
TA_INCLUDE_PATH: ${{ github.workspace }}/dependencies/include
239-
TA_LIBRARY_PATH: ${{ github.workspace }}/dependencies/lib
240-
- name: Upload wheels
241-
uses: actions/upload-artifact@v4
242-
with:
243-
name: wheels-macos-${{ matrix.target }}
244-
path: dist
245-
- name: Test wheel installation
246-
run: |
247-
pip install dist/*.whl
248-
python -c "import polars_talib; print('Import successful')"
249-
250213
macos:
251214
runs-on: macos-latest
252215
strategy:
253216
matrix:
254-
target: [aarch64-apple-darwin] # x86_64,
217+
target: [x86_64, aarch64-apple-darwin]
255218
steps:
256219
- uses: actions/checkout@v4
257220
- uses: actions/setup-python@v5
@@ -275,7 +238,7 @@ jobs:
275238
- name: Upload wheels
276239
uses: actions/upload-artifact@v4
277240
with:
278-
name: wheels-macos-${{ matrix.target }}
241+
name: wheels
279242
path: dist
280243
- name: Test wheel installation
281244
run: |
@@ -286,13 +249,11 @@ jobs:
286249
name: Release
287250
runs-on: ubuntu-latest
288251
if: startsWith(github.ref, 'refs/tags/')
289-
needs: [linux, macos, windows, macos-intel, linux-arm]
252+
needs: [linux, macos, windows, linux-arm]
290253
steps:
291254
- uses: actions/download-artifact@v4
292255
with:
293-
path: dist
294-
pattern: wheels-*
295-
merge-multiple: true
256+
name: wheels
296257
- name: Publish to PyPI
297258
uses: PyO3/maturin-action@v1
298259
env:

0 commit comments

Comments
 (0)