Skip to content

Conversation

hmeriann
Copy link
Contributor

@hmeriann hmeriann commented Jul 8, 2025

bump to v1.3.2

@hmeriann hmeriann marked this pull request as ready for review July 9, 2025 08:37
@samansmink samansmink merged commit ecd8397 into duckdb:main Jul 9, 2025
8 checks passed
[dependencies]
duckdb = { version = "1.3.1", features = ["vtab-loadable"] }
duckdb = { version = "1.3.2", features = ["vtab-loadable"] }
duckdb-loadable-macros = "0.1.5"
Copy link
Member

Choose a reason for hiding this comment

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

I think it'd be a good idea to update this to 0.1.9, so that template users get the latest version of all crates.

Copy link
Member

Choose a reason for hiding this comment

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

We currently release duckdb, duckdb-loadable-macros, and libduckdb-sys together.

Copy link
Collaborator

Choose a reason for hiding this comment

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

oh oops I pressed merged right as you commented!

Copy link
Member

Choose a reason for hiding this comment

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

No problem! 😀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mlafeldt , looks like the version is the same as you've mentioned in generated Cargo.lock file:
https://github.com/hmeriann/extension-template-rs/blob/a2f4c9d0ad1c4427dca799537eea4cb13ea50898/Cargo.lock#L474

So I'll just send a new PR to update duckdb-loadable-macros in the Cargo.toml file. Does it sound right?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, that's right.

It's already the latest version in Cargo.lock because the version constraint 0.1.x allows cargo to update to 0.1.9. However, it's better to make it explicit like with the other duckdb crates.

Approved the PR. :)

Copy link
Contributor Author

@hmeriann hmeriann Jul 9, 2025

Choose a reason for hiding this comment

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

Thanks!
For the future PRs like this, where to look for the proper numbers for the duckdb-loadable-macros?
I'm updating the release check-list and it'd be nice to have this information there (if there any other ways except for running cargo check and looking up for the numbers in generated file)

Copy link
Member

Choose a reason for hiding this comment

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

You're welcome!

I guess https://crates.io/search?q=duckdb&sort=recent-updates at the top would be easiest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

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