Skip to content
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

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

ulrichard
Copy link
Collaborator

No description provided.

@ulrichard ulrichard requested a review from notmandatory July 5, 2024 15:00
tests/multi_sig.rs Outdated Show resolved Hide resolved
tests/regtestenv.rs Outdated Show resolved Hide resolved
@notmandatory notmandatory added the enhancement New feature or request label Jul 12, 2024
@notmandatory
Copy link
Member

To get this to build with rust 1.63 you need to copy/paste or cherry-pick this patch: notmandatory@f85cdcc

@gnapoli23
Copy link

Hi @notmandatory , the fixes should be in. Please check it, and thanks.

@gnapoli23
Copy link

Hello @notmandatory, any update on this? Thank you.

@gnapoli23
Copy link

@notmandatory Reminder to check this please. Thanks.

@ulrichard
Copy link
Collaborator Author

Hey @notmandatory I opened this PR just before going on holiday. Giovanni made all the necessary adjustments. Could you have another look at it?

@notmandatory
Copy link
Member

For some reason the CI didn't run on this PR, can you try re-pushing ?

@ulrichard ulrichard force-pushed the feature/bdk29 branch 4 times, most recently from d83aeb0 to d391fd8 Compare August 21, 2024 13:19
Copy link
Member

@notmandatory notmandatory left a 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.

Makefile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
CI fix for MSRV 1.63

Co-authored-by: Steve Myers <[email protected]>

Co-authored-by: Steve Myers <[email protected]>
Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

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

ACK 3295228

@notmandatory notmandatory merged commit 62ddcb1 into bitcoindevkit:main Aug 24, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants