Followups for getdeploymentinfo #24187

1. 
   ajtowns commented at 7:32 am on January 28, 2022: member

   Documentation, comments and trivial code changes to followup #23508.
2. in doc/release-notes-23508.md:11 in 30572275c4 outdated

    6+  block, rather than just at the chain tip. Inclusion of soft fork
    7+  status in `getblockchaininfo` can currently be restored using the
    8+  configuration `-deprecatedrpc=softforks`, but this will be removed in
    9+  a future release. Note that in either case, the `status` field
   10+  now reflects the status of the current block rather than the next
   11+  block. (#23508)
   

   ---

   ---

   

   MarcoFalke commented at 7:49 am on January 28, 2022:

   could move to the main file to avoid having to do it later? (This should be the last snippet for the next release)
3. 
   MarcoFalke commented at 7:50 am on January 28, 2022: member

   Maybe also #23508 (review) ?
4. ajtowns added this to the milestone 23.0 on Jan 28, 2022
5. doc: getdeploymentinfo release notes tweaks 555eafa793
6. rpc documentation improvements 32f04e6da9
7. trivial: comment tweaks 5179656ef8
8. rpc/blockchain: a constant craving fbab43f169
9. ajtowns force-pushed on Jan 28, 2022
10. 
    ajtowns commented at 8:14 am on January 28, 2022: member

    Was leaving the SoftForkPushBackDesc argument name change ’til other work that would also touch that, but meh, added.
11. MarcoFalke approved
12. 
    MarcoFalke commented at 8:21 am on January 28, 2022: member

    LGTM
13. MarcoFalke added the label Refactoring on Jan 28, 2022
14. 
    luke-jr commented at 3:36 am on January 31, 2022: member

    We haven’t used the variable style behind pindex for years now. Why rename to it?
15. 
    Sjors commented at 2:42 pm on January 31, 2022: member

    We haven’t used the variable style behind pindex for years now. Why rename to it?

    We can name it something else (as long as it’s clear we’re not necessarily dealing with the tip). block_index or just index (I used the latter in the getblockfrompeer RPC code)?

16. rpc/blockchain: rename getdeploymentinfo tip/active_chain_tip to blockindex e5f0356e3f
17. ajtowns force-pushed on Feb 4, 2022
18. 
    ajtowns commented at 3:50 am on February 4, 2022: member

    Changed to blockindex.
19. 
    DrahtBot commented at 5:44 am on February 4, 2022: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #24118 (Add ‘sweepwallet’ RPC by Xekyo)
    • #21702 (Implement BIP-119 Validation (CheckTemplateVerify) by JeremyRubin)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

20. 
    JeremyRubin commented at 6:32 am on February 4, 2022: contributor

    shrug about renaming tip, easy enough to fix the conflict either way.
21. 
    Sjors commented at 8:58 am on February 4, 2022: member

    utACK e5f0356e3ffea10f447998b7549a67e016446e81 @JeremyRubin a rebase beats an off by one soft fork activation error due to unreadable activation-related code :-)
22. MarcoFalke merged this on Feb 14, 2022
23. MarcoFalke closed this on Feb 14, 2022

    ---

24. sidhujag referenced this in commit 2d957724f7 on Feb 14, 2022
25. DrahtBot locked this on Feb 14, 2023

Contributors
ajtowns MarcoFalke luke-jr Sjors DrahtBot JeremyRubin

Labels
Refactoring

Milestone
23.0