qa: Fix some TODOs in p2p_segwit #13626

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1807-qaSegwitTodos changing 1 files +14 −24
  1. MarcoFalke commented at 8:26 PM on July 10, 2018: member
    • I believe we don't need to redundantly test versionbits logic in every functional tests that tests a softfork deployment that is being done with versionbits. Thus, remove two TODOs that ask for that.
    • Replace another TODO with wait_until.
    • Some style fixups after #13467
  2. MarcoFalke added the label Tests on Jul 10, 2018
  3. MarcoFalke requested review from jnewbery on Jul 13, 2018
  4. qa: Fix some TODOs in p2p_segwit eeeef80fb6
  5. MarcoFalke force-pushed on Jul 13, 2018
  6. jnewbery commented at 8:31 PM on July 13, 2018: member

    Tested ACK eeeef80fb69480147e925d8ab2c75f456ff7b1b7. Thanks for tidying up my mess!

  7. MarcoFalke merged this on Jul 13, 2018
  8. MarcoFalke closed this on Jul 13, 2018

  9. MarcoFalke referenced this in commit 8f1106da58 on Jul 13, 2018
  10. MarcoFalke deleted the branch on Jul 13, 2018
  11. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 06:15 UTC

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