Bip classification #192

pull CodeShark wants to merge 2 commits into bitcoin:master from CodeShark:BIP_Classification changing 2 files +407 −0

CodeShark commented at 2:53 PM on September 4, 2015: contributor

Please assign number. BIP123 would be nice.
CodeShark force-pushed on Sep 7, 2015
CodeShark force-pushed on Sep 18, 2015
in bip-layers.mediawiki:None in c79cc3fca5 outdated
```
  64 | +===4. Applications Layer===
  65 | +
  66 | +The applications layer specifies high level structures, abstractions, and conventions that allow different applications to support similar features and share data. 
  67 | +
  68 | +==Classification of existing BIPs==
  69 | +
```
btcdrak commented at 11:23 AM on September 24, 2015:

Why repeat the README.md? This is an unnecessary duplication.

CodeShark commented at 8:41 PM on September 24, 2015:

Which README.md are you referring to?

btcdrak commented at 8:42 PM on September 24, 2015:

meh, your error correction should have inferred my meaning and autocorrected to README.mediawiki :-P

CodeShark commented at 8:46 PM on September 24, 2015:

I still don't follow...I don't see any of this stuff spelled out in the document you refer to.

btcdrak commented at 8:48 PM on September 24, 2015:

It makes no sense to have a section "classification of existing BIPs" when that's a moving target (and already out of date) when your PR could just add the column to the existing README.mediawiki so when it's merged, voilà, and always current.

CodeShark commented at 8:53 PM on September 24, 2015:

The main point was to add a column to that table and set forth specific criteria for assigning BIPs a classification. In the interest of adhering to the BIP process itself, I thought the way to do this was to introduce a new BIP rather than just change the README. Yes, the README is probably the place to ultimately put this info.

luke-jr commented at 8:55 PM on September 24, 2015:

Indeed, just because @CodeShark writes a BIP does not mean it is automatically Accepted.
gmaxwell commented at 12:03 AM on October 22, 2015: contributor

123 assigned.
Added BIP mediawiki 72792b7222
CodeShark force-pushed on Oct 22, 2015
luke-jr added the label New BIP on Oct 23, 2015
luke-jr commented at 12:39 AM on October 23, 2015: member

@CodeShark Add to index?
Added BIP123 to index 5d9dad4277
CodeShark commented at 12:49 AM on October 23, 2015: contributor

@luke-jr Added. The idea is to add the extra column to the table in the README and remove the table from the BIP document once it's accepted.
luke-jr referenced this in commit 96b331e06f on Oct 23, 2015
luke-jr merged this on Oct 23, 2015
luke-jr closed this on Oct 23, 2015
ajtowns referenced this in commit 62486c9572 on Jan 31, 2020

Contributors

Labels