Skip to content

Commit 85105ce

Browse files
Clarify Windows ARM64 wheel testing
Explicitly select ARM64 wheels on the Windows ARM runner and document why its minimal environment still runs the same decoder smoke test as every platform. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 06d16c0 commit 85105ce

2 files changed

Lines changed: 10 additions & 4 deletions

File tree

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ jobs:
3939
os: [ ubuntu-latest, ubuntu-24.04-arm, windows-latest, macos-15-intel, macos-latest ]
4040
include:
4141
- os: windows-11-arm
42+
windows_archs: "ARM64"
4243
before_test: "uv pip install pytest==9.1.1"
4344
test_command: "python -m pytest --confcutdir={project}/tests/avro {project}/tests/avro/test_decoder.py"
4445

@@ -80,11 +81,13 @@ jobs:
8081
env:
8182
# Ignore 32 bit architectures
8283
CIBW_ARCHS: "auto64"
84+
CIBW_ARCHS_WINDOWS: ${{ matrix.windows_archs || 'auto64' }}
8385
CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.15"
8486
# Keep these in sync with Python CI job `cibw-dev-env-smoke-test`
8587
# in .github/workflows/python-ci.yml to catch import-time regressions early.
86-
# PyArrow does not publish Windows ARM64 wheels, so that runner uses
87-
# the minimal test environment configured in the matrix.
88+
# Every platform runs the decoder wheel smoke test. PyArrow does not
89+
# publish Windows ARM64 wheels, so that runner uses a minimal test
90+
# environment and avoids loading the repository-wide conftest.
8891
CIBW_BEFORE_TEST: ${{ matrix.before_test || 'uv sync --directory {project} --only-group dev --no-install-project' }}
8992
CIBW_TEST_COMMAND: ${{ matrix.test_command || 'uv run --directory {project} pytest tests/avro/test_decoder.py' }}
9093
# Skip free-threaded (PEP 703) builds until we evaluate decoder_fast support

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

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ jobs:
3939
os: [ ubuntu-latest, ubuntu-24.04-arm, windows-latest, macos-15-intel, macos-latest ]
4040
include:
4141
- os: windows-11-arm
42+
windows_archs: "ARM64"
4243
before_test: "uv pip install pytest==9.1.1"
4344
test_command: "python -m pytest --confcutdir={project}/tests/avro {project}/tests/avro/test_decoder.py"
4445

@@ -74,9 +75,11 @@ jobs:
7475
env:
7576
# Ignore 32 bit architectures
7677
CIBW_ARCHS: "auto64"
78+
CIBW_ARCHS_WINDOWS: ${{ matrix.windows_archs || 'auto64' }}
7779
CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.15"
78-
# PyArrow does not publish Windows ARM64 wheels, so that runner uses
79-
# the minimal test environment configured in the matrix.
80+
# Every platform runs the decoder wheel smoke test. PyArrow does not
81+
# publish Windows ARM64 wheels, so that runner uses a minimal test
82+
# environment and avoids loading the repository-wide conftest.
8083
CIBW_BEFORE_TEST: ${{ matrix.before_test || 'uv sync --directory {project} --only-group dev --no-install-project' }}
8184
CIBW_TEST_COMMAND: ${{ matrix.test_command || 'uv run --directory {project} pytest tests/avro/test_decoder.py' }}
8285
# Skip free-threaded (PEP 703) builds until we evaluate decoder_fast support

0 commit comments

Comments
 (0)