Skip to content

BIP32: edits by ddustin for clarity#1903

Merged
murchandamus merged 1 commit intobitcoin:masterfrom
jonatack:2025-07-PR785-ddustin-edits-to-BIP32
Mar 5, 2026
Merged

BIP32: edits by ddustin for clarity#1903
murchandamus merged 1 commit intobitcoin:masterfrom
jonatack:2025-07-PR785-ddustin-edits-to-BIP32

Conversation

@jonatack
Copy link
Member

Picks up @ddustin's work in #785, updated with the latest reviewers' feedback.

@jonatack jonatack added Proposed BIP modification Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified labels Jul 25, 2025
@jonatack jonatack changed the title BIP32: edits for clarity BIP32: edits by ddustin for clarity Jul 25, 2025
Copy link
Member

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

Compared to #785 and found to match the old pull request in combination with the outstanding review comments from the BIPs authors except for the mentioned spot.

@jonatack jonatack force-pushed the 2025-07-PR785-ddustin-edits-to-BIP32 branch from 3fa6bc7 to 879066d Compare July 27, 2025 00:07
@murchandamus
Copy link
Member

@sipa: Could you take another look at these few lines?

@murchandamus murchandamus requested a review from sipa February 28, 2026 00:49
@murchandamus murchandamus added the PR Author action required Needs updates, has unaddressed review comments, or is otherwise waiting for PR author label Mar 5, 2026
@jonatack jonatack force-pushed the 2025-07-PR785-ddustin-edits-to-BIP32 branch from 879066d to 1092536 Compare March 5, 2026 19:09
@jonatack jonatack removed the PR Author action required Needs updates, has unaddressed review comments, or is otherwise waiting for PR author label Mar 5, 2026
Co-authored-by: Pieter Wuille <pieter@wuille.net>
Co-authored-by: Murch <murch@murch.one>
Co-authored-by: Jon Atack <jon@atack.com>
@jonatack jonatack force-pushed the 2025-07-PR785-ddustin-edits-to-BIP32 branch from 1092536 to dfdbbd1 Compare March 5, 2026 19:21
Copy link
Member

@sipa sipa left a comment

Choose a reason for hiding this comment

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

ACK dfdbbd1

@murchandamus murchandamus removed the Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified label Mar 5, 2026
Copy link
Member

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

LGTM

@murchandamus murchandamus merged commit c0644a0 into bitcoin:master Mar 5, 2026
4 checks passed
@jonatack jonatack deleted the 2025-07-PR785-ddustin-edits-to-BIP32 branch March 5, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants