-
Couldn't load subscription status.
- Fork 421
Add convenient GossipSync variant constructors #1618
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
Add convenient GossipSync variant constructors #1618
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1618 +/- ##
==========================================
- Coverage 91.07% 90.81% -0.26%
==========================================
Files 80 80
Lines 44128 44645 +517
Branches 44128 44645 +517
==========================================
+ Hits 40190 40546 +356
- Misses 3938 4099 +161
Continue to review full report at Codecov.
|
f3cb61f to
bc08c22
Compare
These constructors fill in the missing types for each variant so that users don't have to turbofish them manually.
bc08c22 to
6877539
Compare
These constructors fill in the missing types for each variant so that users don't have to turbofish them manually.
I verified that we could use all of these within
ldk-samplewithout needing to turbofish.Fixes #1594.