Skip to content

Conversation

hieblmi
Copy link
Contributor

@hieblmi hieblmi commented May 23, 2025

Adds the NoWait flag functional options to CloseChannel.

@hieblmi hieblmi requested review from bhandras and guggero May 23, 2025 10:59
Value lnwire.MilliSatoshi

/// Timestamp of the payment request.
// / Timestamp of the payment request.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: just remove the third character instead?

Copy link
Member

@bhandras bhandras left a comment

Choose a reason for hiding this comment

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

Looks good, just two nits.

@hieblmi hieblmi force-pushed the no-wait branch 3 times, most recently from 0ebc477 to c92c985 Compare May 24, 2025 11:57
@hieblmi hieblmi changed the title closechannel: add NoWait flag closechannel: add functional options May 24, 2025
@hieblmi hieblmi requested review from guggero and bhandras May 24, 2025 11:58
Copy link
Member

@bhandras bhandras left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@guggero guggero merged commit c7caca2 into lightninglabs:master May 26, 2025
1 check 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