Skip to content

Conversation

@tnull
Copy link
Collaborator

@tnull tnull commented Oct 30, 2025

We add a simple workflows to generate the Swift bindings in CI, ensuring the build is succeeding.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Oct 30, 2025

👋 Thanks for assigning @jkczyz 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.

@tnull tnull force-pushed the 2025-10-add-swift-CI branch from e3b06ed to cadd920 Compare October 30, 2025 12:07
tnull added 2 commits October 30, 2025 13:09
We add a simple workflows to generate the Swift bindings in CI, ensuring
the build is succeeding.
.. no need to keep jobs running if we pushed once more.
@tnull tnull force-pushed the 2025-10-add-swift-CI branch from cadd920 to 15a1b90 Compare October 30, 2025 12:13
@tnull tnull requested a review from jkczyz October 30, 2025 12:13
OfferId id();
boolean is_expired();
string? description();
string? offer_description();
Copy link
Contributor

@jkczyz jkczyz Oct 30, 2025

Choose a reason for hiding this comment

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

CI is failing. Looks like some tests need to be updated, e.g.

assert_eq!(ldk_desc.to_string(), wrapped_desc);

Same for the other structs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah, right. Force-pushed fixes.

Swift will auto-generate a `description` accessor. We here avoid any
conflicting methods in our API.
@tnull tnull force-pushed the 2025-10-add-swift-CI branch from 134b2f7 to c554a24 Compare October 31, 2025 12:02
@tnull
Copy link
Collaborator Author

tnull commented Oct 31, 2025

CI failure should be fixed by #689.

@tnull tnull merged commit 8705214 into lightningdevkit:main Oct 31, 2025
0 of 16 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.

3 participants