-
Notifications
You must be signed in to change notification settings - Fork 425
deps(bdk_chain)!: bump miniscript to 13.0.0
#2106
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
deps(bdk_chain)!: bump miniscript to 13.0.0
#2106
Conversation
- 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.
5870b53 to
17d003a
Compare
evanlinjin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 17d003a
|
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 🤔 |
evanlinjin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 17d003a
|
Oh, but how can we ignore them though ? The old jobs are blocking the merge. |
ValuedMammal
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 17d003a
I have special permissions 😉 |
Description
The PR upgrades
miniscriptto it's latestv13.0.0, it's a breaking change so it's only meant to being merged in master, can't be backported torelease/chain-0.23.x.miniscriptversion to13.0.0.psbt.signin `example_cli.BREAKING_CHANGE:
bdk_chaindoes re-exports theminiscriptAPI, therefore bumping it's version is considered a breaking change.Notes to the reviewers
Changelog notice
Checklists
All Submissions:
New Features: