Skip to content

Conversation

@tnull
Copy link
Collaborator

@tnull tnull commented Jul 1, 2025

We bump our uniffi dependency to v0.28.3 to unlock some of the nicer features uniffi added since the previously-used v0.27.3. However, we can't bump it further to v0.29.3, as we have users requiring compatibility with uniffi-bindgen-go, which only supports v0.28.3 at the time of writing.

We bump our `uniffi` dependency to v0.28.3 to unlock some of the nicer
features `uniffi` added since the previously-used v0.27.3. However, we
can't bump it further to v0.29.3, as we have users requiring
compatibility with `uniffi-bindgen-go`, which only supports v0.28.3 at
the time of writing.
@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Jul 1, 2025

I've assigned @tankyleo as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Copy link

@tankyleo tankyleo left a comment

Choose a reason for hiding this comment

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

Not familiar with uniffi myself, reproduced the diff on my machine, things seem to be working, and also ran successfully scripts/uniffi_bindgen_generate_python.sh and scripts/uniffi_bindgen_generate_kotlin.sh.

Likely I am missing some dependencies for scripts/uniffi_bindgen_generate_swift.sh and scripts/uniffi_bindgen_generate_kotlin_android.sh

Also double checked uniffi-bindgen-go and indeed the README says "Currently, uniffi-bindgen-go uses uniffi-rs version 0.28.3".

@tnull tnull merged commit 0a2bccd into lightningdevkit:main Jul 2, 2025
9 of 24 checks passed
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.

4 participants