This pull request forks from #29198 and includes only OP_INTERNALKEY and its tests.
Add OP_INTERNALKEY for Tapscript #29269
pull reardencode wants to merge 1 commits into bitcoin:master from reardencode:ikey-master changing 8 files +44 −4-
reardencode commented at 6:18 AM on January 18, 2024: none
-
DrahtBot commented at 6:18 AM on January 18, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers Concept ACK moonsettler If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #31519 (refactor: Use std::span over Span by maflcko)
- #31460 (fuzz: Expand script verification flag testing to segwit v0 and tapscript by dergoegge)
- #29270 (Implement OP_CHECKSIGFROMSTACK(VERIFY) by reardencode)
- #29247 (CAT in Tapscript (BIP-347) by 0xBEEFCAF3)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
- DrahtBot added the label CI failed on Jan 18, 2024
-
moonsettler commented at 1:14 PM on January 19, 2024: none
Concept ACK!
- glozow added the label Consensus on Feb 5, 2024
-
85206c238d
Add OP_INTERNALKEY for Tapscript
Testing is minimal, but so is the code.
- reardencode force-pushed on Apr 25, 2024
- fanquake closed this on Feb 20, 2025