test: remove unused constants in functional tests #20023

pull theStack wants to merge 1 commits into bitcoin:master from theStack:20200926-test-remove-unused-constants changing 2 files +0 −3
  1. theStack commented at 10:55 AM on September 26, 2020: member

    This mini-PR gets rid of constants in functional tests that are not used anymore. Found by vulture via the following script that has been lying around here locally for quite some time (I think it was once proposed by practicalswift, but I don't remember the concrete topic/PR):

    #!/bin/sh
    for F in $(git ls-files -- "*.py"); do vulture "$F" | grep "unused variable"; done
    
  2. test: remove unused constants in functional tests 92e28fa8b2
  3. DrahtBot added the label Tests on Sep 26, 2020
  4. practicalswift commented at 11:40 AM on September 26, 2020: contributor

    Thanks for cleaning up!

    ACK 92e28fa8b2590cce0e8f0adadae80e46cb63a9ef: patch looks correct.

    Last use of VB_PERIOD in p2p_segwit.py was removed in 0328dcdcfcb56dc8918697716d7686be048ad0b3.

    Last use of FEATURE_LATEST in wallet_multiwallet.py was removed in 0d32d661481f099af572e7a08a50e17bcc165c44.

  5. MarcoFalke merged this on Sep 26, 2020
  6. MarcoFalke closed this on Sep 26, 2020

  7. theStack deleted the branch on Dec 1, 2020
  8. Fabcien referenced this in commit 1b8e36663d on Oct 14, 2021
  9. DrahtBot locked this on Feb 15, 2022
Labels

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

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