Skip to content

Bump LDK Node dependency to commit 9e0cfc5#131

Merged
benthecarman merged 1 commit intolightningdevkit:mainfrom
tnull:2026-02-bump-ldk-node
Feb 25, 2026
Merged

Bump LDK Node dependency to commit 9e0cfc5#131
benthecarman merged 1 commit intolightningdevkit:mainfrom
tnull:2026-02-bump-ldk-node

Conversation

@tnull
Copy link
Collaborator

@tnull tnull commented Feb 24, 2026

No description provided.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Feb 24, 2026

👋 Thanks for assigning @benthecarman 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 requested a review from benthecarman February 24, 2026 08:22
@benthecarman
Copy link
Collaborator

has msrv issues

`9e0cfc5fa9b9dd74fefb795580d00b0a46c8f3a3`

Signed-off-by: Elias Rohrer <dev@tnull.de>
@tnull tnull force-pushed the 2026-02-bump-ldk-node branch from 2030dc9 to 4aef705 Compare February 25, 2026 08:33
@tnull
Copy link
Collaborator Author

tnull commented Feb 25, 2026

has msrv issues

Fixed!

The issues where due to time being used somewhere in the huge dependency tree of lapin. I held off commenting on the introduced dependencies in #128 as they were test-only, but I think they'll come to bite us either way, e.g., due to MSRV conflicts. IMO we should already reconsider how we can drop most of them.

@benthecarman benthecarman merged commit 1a4877b into lightningdevkit:main Feb 25, 2026
7 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