Skip to content

bips 89/114/441: add missing Footnotes and references tag - #2289

Merged
jonatack merged 2 commits into
bitcoin:masterfrom
kriss39:fix/missing-references-tag
Sep 14, 2026
Merged

jonatack merged 2 commits into
bitcoin:masterfrom
kriss39:fix/missing-references-tag

Conversation

@kriss39

@kriss39 kriss39 commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

BIP 89, BIP 114 and BIP 441 use <ref>…</ref> footnotes but none of them contains a <references/> tag. GitHub's mediawiki renderer only emits the footnote bodies where that tag appears, so on the rendered pages the footnote text is missing and the [n] superscripts point at #cite_note-n anchors that do not exist (e.g. BIP 89's notes on lift_x, auxiliary randomness and blindsecnonce; BIP 441's five rationale notes).

This adds a Footnotes section with <references/> to BIP 89 and BIP 441 (before Acknowledgements/Thanks, matching BIP 340), and puts the tag under BIP 114's existing References section.

Rendering-only change, no text of the BIPs is modified, so no changelog entries were added; happy to add one for BIP 89 if preferred.

These three BIPs use <ref> footnotes but never place a <references/>
tag, so GitHub's mediawiki renderer drops the footnote text entirely and
the [n] superscripts link to anchors that do not exist. Add a Footnotes
section (BIP 89, BIP 441) or the tag under the existing References
section (BIP 114) so the notes are rendered.
@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 good fixes, modulo one question below

Comment thread bip-0114.mediawiki
Match the convention used in BIP89 and BIP441: place <references/>
under a dedicated Footnotes section rather than directly under
References, per jonatack's review comment on PR bitcoin#2289.
@kriss39

kriss39 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

Fixed in 50777eb: moved <references/> in bip-0114 under a dedicated Footnotes section, matching the convention used in BIP89 and BIP441.

@jonatack jonatack changed the title bip-89, bip-114, bip-441: add missing <references/> tag bips 89/114/441: add missing Footnotes and references tag Sep 14, 2026
@jonatack

Copy link
Copy Markdown
Member

Thank you for updating.

@jonatack
jonatack merged commit 3a13d60 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