-
Notifications
You must be signed in to change notification settings - Fork 9
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
upgrading to match bdk v29 #36
Conversation
To get this to build with rust 1.63 you need to copy/paste or cherry-pick this patch: notmandatory@f85cdcc |
Hi @notmandatory , the fixes should be in. Please check it, and thanks. |
Hello @notmandatory, any update on this? Thank you. |
@notmandatory Reminder to check this please. Thanks. |
Hey @notmandatory I opened this PR just before going on holiday. Giovanni made all the necessary adjustments. Could you have another look at it? |
For some reason the CI didn't run on this PR, can you try re-pushing ? |
d83aeb0
to
d391fd8
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.
A couple small to fixes and then this looks ready to merge.
CI fix for MSRV 1.63 Co-authored-by: Steve Myers <[email protected]> Co-authored-by: Steve Myers <[email protected]>
d391fd8
to
3295228
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.
ACK 3295228
No description provided.