Skip to content

bip10: Record Withdrawal in Changelog#2171

Merged
jonatack merged 1 commit into
bitcoin:masterfrom
murchandamus:bip10-record-withdrawal
May 21, 2026
Merged

bip10: Record Withdrawal in Changelog#2171
jonatack merged 1 commit into
bitcoin:masterfrom
murchandamus:bip10-record-withdrawal

Conversation

@murchandamus
Copy link
Copy Markdown
Member

BIP10 was withdrawn per #125 after Armory stopped using it and no other projects were known to implement support.

@murchandamus murchandamus added the Metadata Update Changes to Changelog or Preamble without changing the technical content of a BIP. label May 21, 2026
@murchandamus murchandamus force-pushed the bip10-record-withdrawal branch from 414877c to 7571cd4 Compare May 21, 2026 17:54
Comment thread bip-0010.mediawiki

== Changelog ==

* 2014-11-26:
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.

2014-12-09 was the date of merge, if that is preferable

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

If the Changelog entry had been proposed along with the pull request, it would have had the date of when the PR was opened and signed-off by the BIP owner. I don’t think it makes sense for BIP Editors to update a Changelog date before merging a PR to the date of when they get around to merging it — that’d just be busywork. So, the date here feels natural to me as is.

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.

Agree, makes sense. I had understood from other pulls that you were preferring the date of merge, like here: #2167 (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.

Not a big deal either way

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It’s possible that I have not been consistent across my updates in this regard. In that specific case the author had announced a planned date for the merge in advance, though.

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.

True. In some cases, like for activation of a process BIP, or if there is a large difference between dates of opening and merging, it may also make sense to focus on merge date?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sure, in cases where they diverge greatly, merge date seems more reasonable.

Comment thread bip-0010.mediawiki

* 2014-11-26:
** Withdrawn after Armory stopped using BIP10 and no other projects were known to implement support (see [https://github.com/bitcoin/bips/pull/125 bips#125]).
* 2011-10-28:
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, the only commit or pull I found was 4261991 on this date

@jonatack jonatack merged commit 5b118aa into bitcoin:master May 21, 2026
4 checks passed
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