Skip to content

update to bdk v0.19.0 - #5

Merged
ulrichard merged 1 commit into
bitcoindevkit:mainfrom
rajarshimaitra:bdk-update
Jun 17, 2022
Merged

update to bdk v0.19.0#5
ulrichard merged 1 commit into
bitcoindevkit:mainfrom
rajarshimaitra:bdk-update

Conversation

@rajarshimaitra

Copy link
Copy Markdown
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.

@notmandatory notmandatory left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
Collaborator

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

@notmandatory

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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