Skip to content

Conversation

@oleonardolima
Copy link
Collaborator

Description

The PR upgrades miniscript to it's latest v13.0.0, it's a breaking change so it's only meant to being merged in master, can't be backported to release/chain-0.23.x.

  • upgrades miniscript version to 13.0.0.
  • fixes the usage of psbt.sign in `example_cli.

BREAKING_CHANGE: bdk_chain does re-exports the miniscript API, therefore bumping it's version is considered a breaking change.

Notes to the reviewers

Changelog notice

### Changed
- deps(bdk_chain): upgrades `miniscript` version to `13.0.0`.
- chore(example_cli): fixes the usage of `psbt.sign` in `example_cli.

Checklists

All Submissions:

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

@oleonardolima oleonardolima added this to the Chain 0.24.0 milestone Jan 26, 2026
@oleonardolima oleonardolima self-assigned this Jan 26, 2026
@oleonardolima oleonardolima added the dependencies Pull requests that update a dependency file label Jan 26, 2026
@oleonardolima oleonardolima moved this to Needs Review in BDK Chain Jan 26, 2026
- upgrades `miniscript` version to `13.0.0`.
- fixes the usage of `psbt.sign` in `example_cli.
- `miniscript/no-std` has been removed in `13.0.0`, so it's not needed
  anymore to use it in CI.

BREAKING_CHANGE: `bdk_chain` does re-exports the `miniscript` API,
therefore bumping it's version is considered a breaking change.
@oleonardolima oleonardolima force-pushed the deps/bump-miniscript-to-13.x.x branch from 5870b53 to 17d003a Compare January 26, 2026 21:14
Copy link
Member

@evanlinjin evanlinjin left a comment

Choose a reason for hiding this comment

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

ACK 17d003a

@evanlinjin
Copy link
Member

Looks like this needs a rebase on master to get the CI running.

@oleonardolima
Copy link
Collaborator Author

Looks like this needs a rebase on master to get the CI running.

It looks like it's already running fine, the one pending is the old jobs due to the change in this PR 🤔

Copy link
Member

@evanlinjin evanlinjin left a comment

Choose a reason for hiding this comment

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

ACK 17d003a

@oleonardolima
Copy link
Collaborator Author

Oh, but how can we ignore them though ? The old jobs are blocking the merge.

Copy link
Collaborator

@ValuedMammal ValuedMammal left a comment

Choose a reason for hiding this comment

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

ACK 17d003a

@evanlinjin
Copy link
Member

Oh, but how can we ignore them though ? The old jobs are blocking the merge.

I have special permissions 😉

@evanlinjin evanlinjin merged commit 09ddfb2 into bitcoindevkit:master Jan 28, 2026
30 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Chain Jan 28, 2026
@oleonardolima oleonardolima deleted the deps/bump-miniscript-to-13.x.x branch January 31, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file module-blockchain

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants