Skip to content

BIP2: Remove Superseded from choices for Status#458

Merged
luke-jr merged 1 commit intobitcoin:masterfrom
maflcko:patch-3
Oct 5, 2016
Merged

BIP2: Remove Superseded from choices for Status#458
luke-jr merged 1 commit intobitcoin:masterfrom
maflcko:patch-3

Conversation

@maflcko
Copy link
Copy Markdown
Member

@maflcko maflcko commented Oct 5, 2016

The section about the BIP status field only defines
"Replaced or Obsolete (which is equivalent to Replaced)",
so remove "Superseded" for clarity.

The section about the BIP status field only defines
"Replaced or Obsolete (which is equivalent to Replaced)",
so remove "Superseded" for clarity.
@maflcko
Copy link
Copy Markdown
Member Author

maflcko commented Oct 5, 2016

@luke-jr Also the optional Requires header is newly introduced but is not added to the BIP header preamble, so the location/order is not clear. Mind to have a look?

@luke-jr luke-jr merged commit 6399875 into bitcoin:master Oct 5, 2016
@luke-jr
Copy link
Copy Markdown
Member

luke-jr commented Oct 5, 2016

It's not newly introduced, but indeed missing. Added.

@maflcko maflcko deleted the patch-3 branch October 6, 2016 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants