-
Notifications
You must be signed in to change notification settings - Fork 647
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
Ignore: [sBTC] Feat: Peg out request & fulfillment wire formats #3510
Ignore: [sBTC] Feat: Peg out request & fulfillment wire formats #3510
Conversation
Codecov Report
@@ Coverage Diff @@
## 3481-sbtc-peg-in-wire-formats #3510 +/- ##
==================================================================
+ Coverage 31.40% 58.31% +26.91%
==================================================================
Files 305 1 -304
Lines 276271 571 -275700
==================================================================
- Hits 86760 333 -86427
+ Misses 189511 238 -189273 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
14cf605
to
acfdecd
Compare
775576f
to
945b8e7
Compare
db54ddd
to
3710e82
Compare
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.
The only thing which I don't really like is forming SQL queries as strings. Usually, it's a sign of an anti-pattern. However, it's a problem of the base code rather than this PR.
Co-authored-by: Jude Nelson <judecn@gmail.com>
d923e63
to
2ef8209
Compare
Awesome! I've updated & rebased the PR now. I will merge this one once CI passes, it looks like I'm authorized to do so now. Thank you for all the help! 🙏 Edit: I forgot to update the branch. I'm not authorized to merge into |
…eg ops Co-authored-by: Jude Nelson <judecn@gmail.com>
2ef8209
to
1f4594b
Compare
Oh, forgot to point to next... I'll open a new PR to fix it. Edit: Please refer to #3577 for the final merge. |
Introducing wire formats for the peg out request and peg out fulfillment sBTC operations. This PR do not include any signature or sBTC validation for the peg out request. I have created #3514 for this.
I will point this PR to
next
once #3481 is merged.