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

bdk_wallet: Pin bdk_chain version to latest release #1860

Merged

Conversation

evanlinjin
Copy link
Member

@evanlinjin evanlinjin commented Feb 27, 2025

Description

As discussed here: https://discord.com/channels/753336465005608961/753336465005608965/1344807550217883688

Remove the use of Cargo path ref for the bdk_chain and bdk_file_store dependencies. This is the first step to move bdk_wallet into a new workspace.

Also fixed CI failures as we were testing no-std without hashbrown.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@evanlinjin evanlinjin self-assigned this Feb 27, 2025
@evanlinjin evanlinjin force-pushed the pin_chain_version_for_wallet branch from 6bf3f28 to b3904a8 Compare February 28, 2025 00:39
Remove the use of Cargo `path` ref for the `bdk_chain` and
`bdk_file_store` dependencies. This is the first step to move
`bdk_wallet` into a new workspace.
Copy link
Contributor

@LLFourn LLFourn left a comment

Choose a reason for hiding this comment

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

ACK 0c23410

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 0c23410

@notmandatory notmandatory added this to the 1.2.0 milestone Feb 28, 2025
notmandatory added a commit to notmandatory/bdk that referenced this pull request Feb 28, 2025
…test release

0c23410 chore(wallet): Pin `bdk_chain` version to latest release (志宇)
7ce016d chore(examples): pin chain-src crates for wallet examples (志宇)
853dda4 ci: no-std requires hashbrown (志宇)

Pull request description:

  ### Description

  As discussed here: https://discord.com/channels/753336465005608961/753336465005608965/1344807550217883688

  Remove the use of Cargo `path` ref for the `bdk_chain` and `bdk_file_store` dependencies. This is the first step to move `bdk_wallet` into a new workspace.

  Also fixed CI failures as we were testing no-std without hashbrown.

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

ACKs for top commit:
  LLFourn:
    ACK 0c23410
  notmandatory:
    ACK 0c23410

Tree-SHA512: 9c690ae153bc312f045a00ea82bf6772dc3393ee9835d44d0e1ae0570780a84de8cc537471b3c26b438eee49eb3262bd91e246d36e578ff8acd06609f0368dd0
@evanlinjin evanlinjin merged commit 209570d into bitcoindevkit:master Feb 28, 2025
23 checks passed
@nymius nymius mentioned this pull request Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants