Skip to content

Conversation

evertlammerts
Copy link
Contributor

@evertlammerts evertlammerts commented Jul 14, 2025

See https://github.com/duckdblabs/duckdb-internal/issues/5309

This also adds the changes from 0053604 (since we need the version override on both the bugfix and main branches)

@evertlammerts evertlammerts marked this pull request as ready for review July 14, 2025 12:26
@evertlammerts evertlammerts reopened this Jul 14, 2025
Copy link
Contributor

@carlopi carlopi left a comment

Choose a reason for hiding this comment

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

This looks great, I would consider:

"publish_to_pypi": ${{ inputs.should-publish }}

to be set to false to start with, so basically the duckdb-python repo will be notified but will not publish, and once building is tested to work, sending a PR removing Python.yml AND switching this as relevant.

Or directly in this PR skip publishing for Python.yml, but doing so in lockstep, so to avoid both compete for pypi, it should be a matter of avoiding job twine-upload at https://github.com/duckdb/duckdb/blob/main/.github/workflows/Python.yml#L423

@duckdb-draftbot duckdb-draftbot marked this pull request as draft July 14, 2025 14:30
@evertlammerts evertlammerts marked this pull request as ready for review July 14, 2025 14:41
@evertlammerts
Copy link
Contributor Author

Thanks for the review!

This looks great, I would consider:

"publish_to_pypi": ${{ inputs.should-publish }}

to be set to false to start with, so basically the duckdb-python repo will be notified but will not publish, and once building is tested to work, sending a PR removing Python.yml AND switching this as relevant.

Or directly in this PR skip publishing for Python.yml, but doing so in lockstep, so to avoid both compete for pypi, it should be a matter of avoiding job twine-upload at https://github.com/duckdb/duckdb/blob/main/.github/workflows/Python.yml#L423

Right now the workflow in duckdb-python won't publish to pypi, it will just upload to the s3 staging bucket.

@Mytherin Mytherin merged commit e926f28 into duckdb:v1.3-ossivalis Jul 15, 2025
2 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Jul 15, 2025
Backport DB invalidation flag to ossivalis (duckdb/duckdb#18244)
Add nightly builds for out-of-tree python extension (duckdb/duckdb#18239)
Remove require block size from a batch of tests (duckdb/duckdb#18242)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Jul 15, 2025
Backport DB invalidation flag to ossivalis (duckdb/duckdb#18244)
Add nightly builds for out-of-tree python extension (duckdb/duckdb#18239)
Remove require block size from a batch of tests (duckdb/duckdb#18242)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
@evertlammerts evertlammerts deleted the nightly_python_builds branch July 22, 2025 07:45
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.

3 participants