Plus a doc commit.
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-
MarcoFalke commented at 6:08 PM on July 1, 2021: member
-
fa11fecf0d
doc: Move buried deployment doc to the enum that enumerates them
This is more visible than a comment hidden in an RPC helper function.
- MarcoFalke added the label Refactoring on Jul 1, 2021
-
fa5658ed07
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
- MarcoFalke force-pushed on Jul 1, 2021
- fanquake requested review from ajtowns on Jul 2, 2021
-
jnewbery commented at 11:58 AM on July 12, 2021: member
utACK fa5658ed077bfb02b6281d642dc649abdb99b6ee
-
MarcoFalke commented at 5:11 PM on July 12, 2021: member
cc @ajtowns
-
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
- MarcoFalke merged this on Jul 15, 2021
- MarcoFalke closed this on Jul 15, 2021
- MarcoFalke deleted the branch on Jul 15, 2021
- sidhujag referenced this in commit da4e7a1c69 on Jul 23, 2021
- gwillen referenced this in commit ec38fa4592 on Jun 1, 2022
- DrahtBot locked this on Aug 18, 2022