-
Notifications
You must be signed in to change notification settings - Fork 322
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
Backport new taproot descriptor template (BIP86) #1033
Backport new taproot descriptor template (BIP86) #1033
Conversation
This PR solves bitcoindevkit#836. This PR adds a P2TR descriptor template and a BIP86 taproot descriptor template. With this, users can now create a taproot descriptor with templates.
a208d38
to
7587f16
Compare
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.
LGTM!
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.
utACK 7587f16
@vladimirfomene just a heads up, you should explicitly write "ACK <commit_hash>" when reviewing, not just LGTM/approving etc. The reason for this is that we use this script to merge, which records in the merge commit who acked/nacked (see, for example, 8f38e96), and it's important to have all the reviewers and their explicit review in the merge commit (we should rely on git for recording certain info, and not on github)
Summary This patch release backports (from the BDK 1.0 dev branch) a fix for a bug in the policy condition calculation and adds a new taproot single key descriptor template (BIP-86). The policy condition calculation bug can cause issues when a policy subtree fails due to missing info even if it's not selected when creating a new transaction, errors on unused policy paths are now ignored. Fixed - Backported bitcoindevkit#932 fix for policy condition calculation bitcoindevkit#1008 Added - Backported bitcoindevkit#840 taproot descriptor template (BIP-86) bitcoindevkit#1033
Description
This PR solves #836 for the release/0.28 branch. This PR adds a P2TR descriptor template and a BIP86 taproot descriptor template. With this, users can now create a taproot descriptor with templates.
Notes to the reviewers
The commit from #840 is cherry-picked from the
master
branch to therelease/0.28
branch without any changes.Changelog notice
Add taproot descriptor template (BIP-86).
Checklists
All Submissions:
cargo fmt
andcargo clippy
before committingNew Features: