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

update to bdk v0.19.0 #5

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Conversation

rajarshimaitra
Copy link
Contributor

This is an update for migration to bdk v0.19.0 and rust-bitcoin v0.28.

Latest update in bdk has changed few internals and I have updated the tests and the code for them..

We need this crate to be at bdk v0.19.0 so we can move up the same in bdk-cli.

Downstream PR dependent on this: bitcoindevkit/bdk-cli#92

This PR updates with the modifications needed for bdk v0.19.0 and
rust-bitcoin v0.28.1.
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.

utACK 3210861

@ulrichard ulrichard merged commit e19a63f into bitcoindevkit:main Jun 17, 2022
@ulrichard
Copy link
Collaborator

Thanks for updating, before I even realized that 0.19 was released.

@notmandatory
Copy link
Member

Also note that bdk has changed it's MSRV to 1.56.0, and current STABLE rust version to 1.60.0.

@ulrichard
Copy link
Collaborator

Ah yes thanks for the heads up.
I just updated the Readme.
BTW, the link for https://blog.rust-lang.org/2020/08/27/Rust-1.56.0.html in https://github.com/bitcoindevkit/bdk/blob/master/README.md is broken, I used https://blog.rust-lang.org/2021/10/21/Rust-1.56.0.html instead.

@notmandatory
Copy link
Member

Thanks for pointing out the bad link in the README. This PR will fix it: bitcoindevkit/bdk#643

afilini referenced this pull request in bitcoindevkit/bdk Jun 28, 2022
d020ded Fix README.md link to rust 1.56.1 blog post (Steve Myers)

Pull request description:

  ### Description

  Fix link to rust blog for 1.56.1 as pointed out by ulrichard: https://github.com/weareseba/bdk-reserves/pull/5#issuecomment-1159212838

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits

Top commit has no ACKs.

Tree-SHA512: a5bd76fca97dd64c12617b43230dbc36b1178e47224ce324b67cd13999e5f92d2a05d6a9e909841e6d5c6904f2fa426b6bee1001e757d53cc91fb4fd3803f56b
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.

3 participants