Uh oh!
There was an error while loading. Please reload this page.
Update to 0.2.0 and minor tweaks - #141
Conversation
👋 Thanks for assigning @valentinewallace as a reviewer! |
valentinewallace
commented
Sep 29, 2025
CI sad |
TheBlueMatt
commented
Sep 29, 2025
Cause ldk 0.2 isn't uploaded yet :) |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
7c8461b to
009a548Comparee0ad51d to
4961b68Comparevalentinewallace
commented
Dec 17, 2025
I'm good with a squash. Also Windows CI is sad |
ldk-reviews-bot
commented
Dec 18, 2025
🔔 1st Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Dec 22, 2025
🔔 2nd Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Dec 24, 2025
🔔 3rd Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Dec 27, 2025
🔔 4th Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Dec 29, 2025
🔔 5th Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Dec 31, 2025
🔔 6th Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Jan 3, 2026
🔔 7th Reminder Hey @valentinewallace! This PR has been waiting for your review. |
ldk-reviews-bot
commented
Jan 5, 2026
🔔 8th Reminder Hey @valentinewallace! This PR has been waiting for your review. |
valentinewallace
left a comment
There was a problem hiding this comment.
Can we squash + fix CI? Should be good to go
Naively update to LDK 0.2 using the sync wrappers to keep current behavior.
With LDK 0.2, most traits now offer async variants. In the previous commit we ignored these but here we migrate to them everywhere except in the `ChannelMonitor` persistence. Note that we have to switch to async stdin reading in cli.rs to ensure we always reach an `await` point in a timely manner on all tasks.
Now that we have `ChainMonitor::load_existing_monitor` we can use it during startup to avoid the excess persistence at startup.
Its always been a bit strange that we persist the addresses of our peers to disk when there's a public gossip db, its not really a thing most LN nodes do. It might be useful for private channels but there's not a lot of reason to have it in `ldk-sample`.
TheBlueMatt
commented
Jan 6, 2026
Sorry for the delay, squashed and reverted the extra changes to |
valentinewallace
left a comment
There was a problem hiding this comment.
LGTM. Looks like Windows 1.63 CI is still sad
TheBlueMatt
commented
Jan 15, 2026
Not sure we care that much about windows-MSRV builds, so just disabled it. |
TheBlueMatt
commented
Feb 3, 2026
THe windows build is some windows crap. 1.63 didn't fail, it just was cancelled when windows did a windows thing. |
Uh oh!
There was an error while loading. Please reload this page.
No description provided.