Skip to content

bip1: Record close reason#2167

Merged
jonatack merged 1 commit into
bitcoin:masterfrom
murchandamus:bip1-record-close-reason
May 21, 2026
Merged

bip1: Record close reason#2167
jonatack merged 1 commit into
bitcoin:masterfrom
murchandamus:bip1-record-close-reason

Conversation

@murchandamus
Copy link
Copy Markdown
Member

This commit forgoes introducing a version, as the BIP has been closed for a decade and it is no longer necessary to update the readers regarding specification changes of BIP1.

This commit forgoes introducing a version, as the BIP has been closed
for a decade and it is no longer necessary to update the readers
regarding specification changes of BIP1.
@murchandamus murchandamus added the Metadata Update Changes to Changelog or Preamble without changing the technical content of a BIP. label May 21, 2026
Copy link
Copy Markdown
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK, modulo that #478 updated BIP 1 from Active to Replaced, and then 66defbd updated BIP 1 from Replaced to Closed. Do you think it is worth describing this with separate changelog entries? I think it may be clearer to readers.

Comment thread bip-0001.mediawiki
10 Oct 2015 - Added clarifications about submission process and BIP number assignment.

01 Jan 2016 - Clarified early stages of BIP idea championing, collecting community feedback, etc.
* 2016-12-14:
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.

Indeed, corresponds to the date #478 was merged.

@murchandamus
Copy link
Copy Markdown
Member Author

ACK, modulo that #478 updated BIP 1 from Active to Replaced, and then 66defbd updated BIP 1 from Replaced to Closed. Do you think it is worth describing this with separate changelog entries? I think it may be clearer to readers.

I don’t think that it adds value to record the Deployment of BIP3 in every closed BIP’s Changelog. Therefore, my preferred approach would be to fold the update to Closed into the Changelog entry that records the original closure, while recording the close reason in that context to reflect what status update was made originally. In this case, "superseded" conveys that BIP1 was replaced.

@jonatack jonatack merged commit 242f7d2 into bitcoin:master May 21, 2026
4 checks passed
@jonatack
Copy link
Copy Markdown
Member

While it seems somewhat more true to history to record the change, it's not a strong opinion and point taken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Metadata Update Changes to Changelog or Preamble without changing the technical content of a BIP.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants