Skip to content

Conversation

@tnull
Copy link
Collaborator

@tnull tnull commented Jan 13, 2026

Just a tiny follow-up to #750.

Previously we introduced a create_test_entropy helper method in common.rs that was only used in one specific test. We here opt to drop this again and also clean up the remaining warnings when run under --features uniffi.

Previously we introduced a `create_test_entropy` helper method in
`common.rs` that was only used in one specific test. We here opt to drop
this again and also clean up the remaining warnings when run under
`--features uniffi`.
@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Jan 13, 2026

I've assigned @joostjager 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
Contributor

@joostjager joostjager left a comment

Choose a reason for hiding this comment

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

Should uniffi warnings fail CI?

@tnull
Copy link
Collaborator Author

tnull commented Jan 13, 2026

Should uniffi warnings fail CI?

Hmm, I don't think it's that important.

@tnull tnull merged commit c6903f6 into lightningdevkit:main Jan 13, 2026
17 of 18 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