-
Notifications
You must be signed in to change notification settings - Fork 339
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace InvoicePayer
with utility methods
#2007
Replace InvoicePayer
with utility methods
#2007
Conversation
2f0978c
to
026bceb
Compare
Codecov ReportBase: 90.89% // Head: 91.07% // Increases project coverage by
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #2007 +/- ##
==========================================
+ Coverage 90.89% 91.07% +0.17%
==========================================
Files 99 99
Lines 53026 51731 -1295
Branches 53026 51731 -1295
==========================================
- Hits 48197 47113 -1084
+ Misses 4829 4618 -211
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
026bceb
to
ac0c461
Compare
ac0c461
to
1f6c88a
Compare
1f6c88a
to
bc9ccfc
Compare
Fixed bad doc link
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good from a first review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM feel free to squash. I think we need to migrate considers_inflight_htlcs_between_retries
and but that can come in a followup.fails_retrying_invoice_after_expiration
b7c5e47
to
a23b5eb
Compare
a23b5eb
to
f78448c
Compare
We're moving payment retries to
ChannelManager
in preparation for supporting trampoline payments.Partially addresses #1932
Based on #2002 with another small commit taken from #1996