No description provided.
Updated C++ library link and added acknowledgement. #218
pull CodeShark wants to merge 1 commits into bitcoin:master from CodeShark:link_and_ack changing 1 files +2 −1-
CodeShark commented at 7:59 PM on October 9, 2015: contributor
- CodeShark force-pushed on Oct 9, 2015
-
jonasschnelli commented at 8:15 PM on October 9, 2015: contributor
ACK
-
sipa commented at 8:17 PM on October 9, 2015: member
I really hate to nitpick about this, but I don't think I can acknowledge you in a BIP for an implementation that didn't exist at the time.
-
CodeShark commented at 8:23 PM on October 9, 2015: contributor
I had an implementation way before Bitcoin Core did...and the implementation has been listed in this BIP document forever. I'll take that part out if it bugs you - it's more important that the link to the implementation be there. .
-
sipa commented at 8:30 PM on October 9, 2015: member
Sure, but not when the BIP was written...
- CodeShark force-pushed on Oct 9, 2015
-
Updated C++ library link and added acknowledgement. 38ea34dee5
- luke-jr added the label Proposed BIP modification on Oct 23, 2015
- luke-jr assigned sipa on Oct 23, 2015
-
-
sipa commented at 6:10 PM on January 8, 2016: member
ACK
-
sipa commented at 6:24 PM on January 8, 2016: member
I have no authority over the bips repository, nor is there any reason why I should. The fact that it is hosted under the bitcoin organization is coincidental, so I do not use this to merge bip changes.
- luke-jr referenced this in commit 01c5e719d3 on Jan 18, 2016
- luke-jr merged this on Jan 18, 2016
- luke-jr closed this on Jan 18, 2016
Contributors