Skip to content

Bump bdk version to 1.0.0-alpha.10#1420

Merged
danielabrozzoni merged 1 commit intobitcoindevkit:masterfrom
danielabrozzoni:bump_dev_1.0.0_alpha.10
May 2, 2024
Merged

Bump bdk version to 1.0.0-alpha.10#1420
danielabrozzoni merged 1 commit intobitcoindevkit:masterfrom
danielabrozzoni:bump_dev_1.0.0_alpha.10

Conversation

@danielabrozzoni
Copy link
Copy Markdown
Member

@danielabrozzoni danielabrozzoni commented May 2, 2024

Description

bdk_chain to 0.13.0
bdk_bitcoind_rpc to 0.9.0
bdk_electrum to 0.12.0
bdk_esplora to 0.12.0
bdk_file_store to 0.10.0
bdk_testenv to 0.3.0
bdk_persist to 0.2.0

Checklists

All Submissions:

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

@danielabrozzoni danielabrozzoni mentioned this pull request May 2, 2024
3 tasks
danielabrozzoni added a commit that referenced this pull request May 2, 2024
a5fb7fd fix: Cargo clippy lints after rust 1.78 (Daniela Brozzoni)

Pull request description:

  <!-- You can erase any parts of this template not applicable to your Pull Request. -->

  ### Description

  Caught when trying to release (#1420), clippy failed randomly although it worked on master, this happened because rust 1.78 had just been release and we use clippy stable. IMHO we should pin the clippy version in CI and bump it manually at each new rust release.

  ### 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:
  notmandatory:
    ACK a5fb7fd

Tree-SHA512: c803366367576224f9e9690cdee2c0161fc083550355415f9174e93ada2f597440f54ac966bb3ebecdc916824d43de17ac72801e4ef0f75c8a1df640fe40df6d
@danielabrozzoni danielabrozzoni force-pushed the bump_dev_1.0.0_alpha.10 branch from 3c01d79 to d2e0f64 Compare May 2, 2024 13:44
Comment thread crates/file_store/Cargo.toml Outdated
bdk_chain = { path = "../chain", version = "0.12.0", features = [ "serde", "miniscript" ] }
bdk_persist = { path = "../persist", version = "0.1.0"}
bdk_chain = { path = "../chain", version = "0.13.0", features = [ "serde", "miniscript" ] }
bdk_persist = { path = "../persist", version = "0.2.0"}
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.

Suggested change
bdk_persist = { path = "../persist", version = "0.2.0"}
bdk_persist = { path = "../persist", version = "0.1.0"}

Comment thread crates/persist/Cargo.toml Outdated
name = "bdk_persist"
homepage = "https://bitcoindevkit.org"
version = "0.1.0"
version = "0.2.0"
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.

🔧 Don't need to bump this one since it's the first release.

Suggested change
version = "0.2.0"
version = "0.1.0"

@notmandatory
Copy link
Copy Markdown
Member

Everything looks good, just one small change and can be merged.

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

Needs those couple changes but ACKing now so you can merge when you're ready.

bdk_chain to 0.13.0
bdk_bitcoind_rpc to 0.9.0
bdk_electrum to 0.12.0
bdk_esplora to 0.12.0
bdk_file_store to 0.10.0
bdk_testenv to 0.3.0
bdk_persist to 0.2.0
@danielabrozzoni danielabrozzoni force-pushed the bump_dev_1.0.0_alpha.10 branch from d2e0f64 to 2cda9f4 Compare May 2, 2024 15:34
Copy link
Copy Markdown

@storopoli storopoli left a comment

Choose a reason for hiding this comment

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

ACK 2cda9f4

@danielabrozzoni danielabrozzoni merged commit fbe1782 into bitcoindevkit:master May 2, 2024
@danielabrozzoni danielabrozzoni deleted the bump_dev_1.0.0_alpha.10 branch May 2, 2024 15:54
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