Skip to content

ci: fix nightly and release#3215

Merged
kevinjqliu merged 6 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-nightly
Mar 31, 2026
Merged

ci: fix nightly and release#3215
kevinjqliu merged 6 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-nightly

Conversation

@kevinjqliu
Copy link
Copy Markdown
Contributor

Rationale for this change

Follow up to #3193
env vars does not play nice with windows os (because the default shell is powershell). We need to set to bash so that env vars are interpreted correctly

Are these changes tested?

Yes
Ran on fork
nightly: https://github.com/kevinjqliu/iceberg-python/actions/runs/23805622644
release: https://github.com/kevinjqliu/iceberg-python/actions/runs/23805619712

Are there any user-facing changes?

Copy link
Copy Markdown
Member

@geruh geruh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non blocking: do we need to go and update the svn-build-artifacts.yml since it uses windows-latest.

jobs:
set-version:
if: github.repository == 'apache/iceberg-python' # Only run for apache repo
if: github.repository == 'apache/iceberg-python' || github.event_name == 'workflow_dispatch'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice add!

@kevinjqliu
Copy link
Copy Markdown
Contributor Author

non blocking: do we need to go and update the svn-build-artifacts.yml since it uses windows-latest.

nope i dont think so, the only env: block in that file is for uses: pypa/cibuildwheel, not run: like the others

env:
# Ignore 32 bit architectures
CIBW_ARCHS: "auto64"
CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<3.14"
CIBW_BEFORE_TEST: "uv sync --directory {project} --only-group dev --no-install-project"
CIBW_TEST_COMMAND: "uv run --directory {project} pytest tests/avro/test_decoder.py"
# Skip free-threaded (PEP 703) builds until we evaluate decoder_fast support
CIBW_SKIP: "cp3*t-*"

@kevinjqliu kevinjqliu merged commit 39e08a1 into apache:main Mar 31, 2026
6 checks passed
@kevinjqliu kevinjqliu deleted the kevinjqliu/fix-nightly branch March 31, 2026 17:14
@kevinjqliu
Copy link
Copy Markdown
Contributor Author

kevinjqliu commented Mar 31, 2026

nice! nightly is back https://github.com/apache/iceberg-python/actions/runs/23810165138

thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants