Skip to content

Conversation

@darosior
Copy link
Contributor

@darosior darosior commented Aug 4, 2021

Y and Z were mixed up

@sanket1729
Copy link
Member

rustfmt failure, and looks like our earlier tests were broken

@darosior darosior force-pushed the fix_lift branch 3 times, most recently from 8a450ee to ef17f0f Compare August 4, 2021 21:26
@darosior
Copy link
Contributor Author

darosior commented Aug 4, 2021

Fixed rustfmt and 1.29 compilation

Y and Z were mixed up

Signed-off-by: Antoine Poinsot <[email protected]>
@darosior
Copy link
Contributor Author

darosior commented Aug 4, 2021

Fixed the HTLC example, which asserted the wrong behaviour.

Copy link
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 7304792

@sanket1729 sanket1729 merged commit 3efe384 into rust-bitcoin:master Aug 5, 2021
@darosior darosior deleted the fix_lift branch August 5, 2021 07:41
danielabrozzoni added a commit to danielabrozzoni/revaultd that referenced this pull request Aug 5, 2021
Manager threshold support needs a new revault_tx release
since rust-bitcoin/rust-miniscript#261
This commits only lays out the structure for the support, another
commit containing the actual threshold calculation will be needed.
danielabrozzoni added a commit to danielabrozzoni/revaultd that referenced this pull request Aug 5, 2021
Manager threshold support needs a new revault_tx release
since rust-bitcoin/rust-miniscript#261
This commits only lays out the structure for the support, another
commit containing the actual threshold calculation will be needed.
danielabrozzoni added a commit to danielabrozzoni/revaultd that referenced this pull request Aug 7, 2021
Manager threshold support needs a new revault_tx release
since rust-bitcoin/rust-miniscript#261
This commits only lays out the structure for the support, another
commit containing the actual threshold calculation will be needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants