Fix wrong test vector in BIP-388. Sometimes /<0;1>/* is missing. Sometimes it is incorrectly written as <0,1>. #1694

pull pad01g wants to merge 1 commits into bitcoin:master from pad01g:bip-388-test-vectors-valid-policies changing 1 files +8 −9
  1. pad01g commented at 9:27 am on November 5, 2024: contributor
    I found that test vectors in BIP-388 contain errors regarding multipath descriptor derivation. Also, fixed invalid newline in multipath descriptor derivation doc.
  2. fix wrong test vector. sometimes /<0;1>/* is missing. sometimes incorrectly written as <0,1>. dcb140a865
  3. pad01g marked this as ready for review on Nov 5, 2024
  4. jonatack added the label Proposed BIP modification on Nov 5, 2024
  5. jonatack added the label Pending acceptance on Nov 5, 2024
  6. jonatack commented at 6:05 pm on November 5, 2024: member
    cc BIP author @bigspider
  7. bigspider commented at 8:38 am on November 6, 2024: contributor

    ACK

    Thanks for the fix, @pad01g!

  8. jonatack removed the label Pending acceptance on Nov 6, 2024
  9. jonatack commented at 7:25 pm on November 7, 2024: member
    ACK
  10. jonatack merged this on Nov 7, 2024
  11. jonatack closed this on Nov 7, 2024

  12. jonatack added the label Bug fix on Nov 8, 2024
  13. jonatack removed the label Proposed BIP modification on Nov 8, 2024
  14. pad01g deleted the branch on Dec 13, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-21 16:10 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me