-
Notifications
You must be signed in to change notification settings - Fork 14
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
v28: Add support for submitpackage
#14
Conversation
cc @tnull |
4184d7f
to
3e17653
Compare
integration_test/src/lib.rs
Outdated
/// | ||
/// # Returns | ||
/// | ||
/// The receive address and the transaction. |
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.
In 3e17653:
Trailing whitespace.
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.
Were are the git hooks!!!
3e17653
to
b676790
Compare
Otherwise 3e17653 looks good. Also CI is failing. I didn't look at why. |
Bitcoin Core introduced a new RPC method `submitpackage`. Add support for it including integration testing. This work was originally done by tnull a while ago over on the old `rust-bitcoind-json-rpc` repository (PR: rust-bitcoin#23). I took his changes and updated the code with the ideas I've learned recently fleshing out v17. <Co-developed-by>: Elias Rohrer <dev@tnull.de>
b676790
to
7c3024c
Compare
Thanks man, CI was docs. Should be fixed now. |
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.
ACK 7c3024c; successfully ran local tests
The PR description is wrong - |
Sorry! 😁 |
No stress man, shit happens. |
Bitcoin Core introduced a new RPC method
submitpackage
. Add support for it including integration testing.This work was originally done by tnull a while ago over on the old
rust-bitcoind-json-rpc
repository (PR: rust-bitcoin/rust-bitcoind-json-rpc#23).I took his changes and updated the code with the ideas I've learned recently fleshing out v17.
Patch includes
<Co-developed-by>
tag.Close: #7