Skip to content

Commit f195dad

Browse files
authored
Clean up cibuildwheel build (apache#1599)
Removes `CIBW_TEST_EXTRAS: "s3fs,glue" since we dont need the extra dependencies for tests Removes skipping MacOS build in `CIBW_TEST_SKIP` Test run on fork of release candidate on https://github.com/kevinjqliu/iceberg-python/actions/runs/13117565428
1 parent f2b42e5 commit f195dad

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

.github/workflows/pypi-build-artifacts.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,10 @@ jobs:
7575
CIBW_ARCHS: "auto64"
7676
CIBW_PROJECT_REQUIRES_PYTHON: ">=3.9,<3.13"
7777
CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1"
78-
CIBW_TEST_EXTRAS: "s3fs,glue"
7978
CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py"
80-
# There is an upstream issue with installing on MacOSX
81-
# https://github.com/pypa/cibuildwheel/issues/1603
8279
# Ignore tests for pypy since not all dependencies are compiled for it
8380
# and would require a local rust build chain
84-
CIBW_TEST_SKIP: "pp* *macosx*"
81+
CIBW_TEST_SKIP: "pp*"
8582

8683
- name: Add source distribution
8784
if: startsWith(matrix.os, 'ubuntu')

.github/workflows/svn-build-artifacts.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,10 @@ jobs:
6969
CIBW_ARCHS: "auto64"
7070
CIBW_PROJECT_REQUIRES_PYTHON: ">=3.9,<3.13"
7171
CIBW_TEST_REQUIRES: "pytest==7.4.2 moto==5.0.1"
72-
CIBW_TEST_EXTRAS: "s3fs,glue"
7372
CIBW_TEST_COMMAND: "pytest {project}/tests/avro/test_decoder.py"
74-
# There is an upstream issue with installing on MacOSX
75-
# https://github.com/pypa/cibuildwheel/issues/1603
7673
# Ignore tests for pypy since not all dependencies are compiled for it
7774
# and would require a local rust build chain
78-
CIBW_TEST_SKIP: "pp* *macosx*"
75+
CIBW_TEST_SKIP: "pp*"
7976

8077
- name: Add source distribution
8178
if: startsWith(matrix.os, 'ubuntu')

0 commit comments

Comments
 (0)