-
Notifications
You must be signed in to change notification settings - Fork 407
Introduce bLIP numbers in lightning-liquidity
documentation
#3510
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
Introduce bLIP numbers in lightning-liquidity
documentation
#3510
Conversation
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.
Thanks.
Feel free to squash. |
0fd3c19
to
4f077b9
Compare
Done diff --git a/lightning-liquidity/src/lsps2/client.rs b/lightning-liquidity/src/lsps2/client.rs
index 70881de6f..9f56f98a2 100644
--- a/lightning-liquidity/src/lsps2/client.rs
+++ b/lightning-liquidity/src/lsps2/client.rs
@@ -62,5 +62,5 @@ impl PeerState {
/// Note that currently only the 'client-trusts-LSP' trust model is supported, i.e., we don't
/// provide any additional API guidance to allow withholding the preimage until the channel is
-/// opened. Please refer to the [`LSPS2 / blIP-52 specification`] for more information.
+/// opened. Please refer to the [`LSPS2 / bLIP-52 specification`] for more information.
///
/// [`LSPS2 / bLIP-52 specification`]: https://github.com/lightning/blips/blob/master/blip-0052.md#trust-models |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3510 +/- ##
==========================================
+ Coverage 88.30% 90.46% +2.16%
==========================================
Files 149 149
Lines 112921 130229 +17308
Branches 112921 130229 +17308
==========================================
+ Hits 99718 117816 +18098
+ Misses 10722 10071 -651
+ Partials 2481 2342 -139 ☔ View full report in Codecov by Sentry. |
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.
Grr
4f077b9
to
761fbb0
Compare
Addressed the comment, and also globally switched |
761fbb0
to
92424be
Compare
Recently, LSPS0, 1, and 2 were upstreamed as bLIP-50, 51, and 52, respectively. Here, we 1. start linking to the bLIPs instead of the LSP spec repository, which is likely going to be deprecated. 2. start consistently citing the specs as `bLIP-5X / LSPSX` to avoid any confusions and to potentially initiate a process in which the LSP specs will be referred to by their bLIP number rather than their LSPS number (especially given that any upcoming specs by the LSP spec group will directly be drafted as bLIPs going forward).
92424be
to
09a79c2
Compare
Just doc changes, so gonna land. |
Recently, LSPS0, 1, and 2 were upstreamed as bLIP-50, 51, and 52, respectively.
Here, we
bLIP-5X / LSPSX
to avoid any confusions and to potentially initiate a process in which the LSP specs will over time get referred to by their bLIP number rather than their LSPS number (especially given that any upcoming specs by the LSP spec group will directly be drafted as bLIPs going forward).