Skip to content

bip-433: fix heading levels, stray '<', typos, add Copyright section - #2291

Merged
jonatack merged 1 commit into
bitcoin:masterfrom
kriss39:fix/bip433-formatting
Sep 14, 2026
Merged

jonatack merged 1 commit into
bitcoin:masterfrom
kriss39:fix/bip433-formatting

Conversation

@kriss39

@kriss39 kriss39 commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Editorial fixes to BIP 433 (P2A), no change to the specification itself:

  • ===Specification=== was a level-3 heading, so on the rendered page (and in the TOC) Specification appears as a subsection of Motivation; Implementation/Related Work were level 4 and Backward Compatibility was level 4 instead of being a top-level section. Reset them to ==Specification== / ===Implementation=== / ===Related Work=== / ==Backward Compatibility==.
  • Remove the stray < in <code><tb1pfees9rn5nz</code> (the address itself is correct: bech32m("tb", 1, 0x4e73) = tb1pfees9rn5nz).
  • Typos: "output typea" → "output type", "ineherent" → "inherent".
  • Add the Copyright section that BIP 3 requires, stating the same BSD-3-Clause terms as the License header.

- Specification was a level-3 heading and therefore rendered as a
  subsection of Motivation; Backward Compatibility, Implementation and
  Related Work were one level too deep as well.
- Drop the stray '<' in front of the testnet address tb1pfees9rn5nz.
- 'typea' -> 'type', 'ineherent' -> 'inherent'.
- Add the Copyright section required by BIP 3, matching the
  BSD-3-Clause License header.
@jonatack jonatack added the Fixups Minor fixups not worth bothering the BIP author(s) for label Sep 14, 2026

@jonatack jonatack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK

@jonatack
jonatack merged commit aeff236 into bitcoin:master Sep 14, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fixups Minor fixups not worth bothering the BIP author(s) for

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants