-
Notifications
You must be signed in to change notification settings - Fork 28
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add python wheel builds and enable more platforms (#183)
* Add python wheel artifacts and enable other platforms * Disable aarch 64 for now * Attempt to fix aarch build * Add doc * Adjust manylinux for all ubuntu targets * plzzz * Simplify test runners * Add new ci flow with only tests for pushes, and a special release flow
- Loading branch information
1 parent
65f698c
commit 626b34b
Showing
2 changed files
with
192 additions
and
207 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,75 @@ | ||
name: Python Check | ||
|
||
on: | ||
push: | ||
branches: | ||
- main | ||
pull_request: | ||
workflow_dispatch: | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.ref }} | ||
cancel-in-progress: true | ||
|
||
permissions: | ||
contents: read | ||
|
||
defaults: | ||
run: | ||
working-directory: ./icechunk-python | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Stand up MinIO | ||
run: | | ||
docker compose up -d minio | ||
- name: Wait for MinIO to be ready | ||
run: | | ||
for i in {1..10}; do | ||
if curl --silent --fail http://minio:9000/minio/health/live; then | ||
break | ||
fi | ||
sleep 3 | ||
done | ||
docker compose exec -T minio mc alias set minio http://minio:9000 minio123 minio123 | ||
- uses: actions/setup-python@v5 | ||
with: | ||
python-version: '3.11' | ||
- name: Build wheels | ||
uses: PyO3/maturin-action@v1 | ||
with: | ||
working-directory: icechunk-python | ||
target: ${{ matrix.platform.target }} | ||
args: --release --out dist --find-interpreter | ||
sccache: 'true' | ||
manylinux: ${{ matrix.platform.manylinux }} # https://github.com/PyO3/maturin-action/issues/245 | ||
- name: mypy | ||
shell: bash | ||
working-directory: icechunk-python | ||
run: | | ||
set -e | ||
python3 -m venv .venv | ||
source .venv/bin/activate | ||
pip install icechunk['test'] --find-links dist --force-reinstall | ||
mypy python | ||
- name: ruff | ||
shell: bash | ||
working-directory: icechunk-python | ||
run: | | ||
set -e | ||
python3 -m venv .venv | ||
source .venv/bin/activate | ||
pip install icechunk['test'] --find-links dist --force-reinstall | ||
ruff check | ||
- name: pytest | ||
shell: bash | ||
working-directory: icechunk-python | ||
run: | | ||
set -e | ||
python3 -m venv .venv | ||
source .venv/bin/activate | ||
pip install icechunk['test'] --find-links dist --force-reinstall | ||
pytest |
Oops, something went wrong.