refactor: Use DeploymentEnabled to hide VB deployments #22385

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2107-dep changing 3 files +10 −11
  1. MarcoFalke commented at 6:08 PM on July 1, 2021: member

    Plus a doc commit.

  2. doc: Move buried deployment doc to the enum that enumerates them
    This is more visible than a comment hidden in an RPC helper function.
    fa11fecf0d
  3. MarcoFalke added the label Refactoring on Jul 1, 2021
  4. Use DeploymentEnabled to hide VB deployments
    The helper was previously unused. This commit changes it to be more
    meaningful and puts it to use.
    
    See previous discussion at https://github.com/bitcoin/bitcoin/pull/19438/files#r650687320
    fa5658ed07
  5. MarcoFalke force-pushed on Jul 1, 2021
  6. fanquake requested review from ajtowns on Jul 2, 2021
  7. jnewbery commented at 11:58 AM on July 12, 2021: member

    utACK fa5658ed077bfb02b6281d642dc649abdb99b6ee

  8. MarcoFalke commented at 5:11 PM on July 12, 2021: member
  9. ajtowns commented at 1:22 AM on July 13, 2021: member

    utACK fa5658ed077bfb02b6281d642dc649abdb99b6ee

    Looks good to me. Earlier discussion was in a comment to 19438

  10. MarcoFalke merged this on Jul 15, 2021
  11. MarcoFalke closed this on Jul 15, 2021

  12. MarcoFalke deleted the branch on Jul 15, 2021
  13. sidhujag referenced this in commit da4e7a1c69 on Jul 23, 2021
  14. gwillen referenced this in commit ec38fa4592 on Jun 1, 2022
  15. DrahtBot locked this on Aug 18, 2022


ajtowns


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:14 UTC

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