Package the Sidequery PyIceberg distribution - #3
Merged
nicosuave merged 1 commit intoAug 18, 2026
Merged
Conversation
nicosuave
force-pushed
the
sidequery-pyiceberg-distribution
branch
3 times, most recently
from
August 18, 2026 05:50
ad01864 to
a726c2e
Compare
nicosuave
force-pushed
the
sidequery-pyiceberg-distribution
branch
from
August 18, 2026 06:20
a726c2e to
6fd7fe6
Compare
nicosuave
marked this pull request as ready for review
August 18, 2026 06:22
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
nicosuave
merged commit Aug 18, 2026
b411581
into
bounded-equality-delete-read-write
15 checks passed
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
sidequery-pyicebergdistribution namepyicebergimport package and CLI0.12.0.post1Why
PyPI rejects VCS dependencies in uploaded package metadata, so
dlt-icebergcannot be released while it directly depends on the GitHub fork. A normal PyPI distribution gives it a publishable dependency without pretending this fork is an Apache PyIceberg release.Validation
uv build --sdist --wheel[pyiceberg-core]import pyiceberg, version metadata, andTable.upsert_by_equality_deleteuv run pytest tests/test_version.py tests/cli/test_console.py -q(87 passed)Before publishing
Configure a PyPI pending Trusted Publisher for:
sidequeryiceberg-pythonsidequery-pypi.ymlpypiThis PR does not publish a package.