Skip to content

Conversation

@theStack
Copy link
Contributor

@theStack theStack commented Nov 1, 2022

These constants exist since the introduction of the functional test wallet_taproot.py (2667366), but they have never been used.

These constants exist since the introduction of the functional test
wallet_taproot.py (2667366), but they
have never been used.
@fanquake fanquake added the Tests label Nov 1, 2022
@instagibbs
Copy link
Member

ACK f1ee974

Copy link
Contributor

@brunoerg brunoerg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK f1ee974

I was able to check it by using git grep

maflcko pushed a commit that referenced this pull request Nov 1, 2022
f1ee974 test: remove unused `CHANGE_{XPRV,XPUB}` constants (Sebastian Falbesoner)

Pull request description:

  These constants exist since the introduction of the functional test wallet_taproot.py (2667366), but they have never been used.

ACKs for top commit:
  instagibbs:
    ACK f1ee974
  brunoerg:
    ACK f1ee974

Tree-SHA512: 2c6f8cd537c35af59c4cbe6abb07386b7fb9e4662c7bed27ad8935e004f9bf452705fb1caef4f5ea1f4ad509745afba966005f95a18cf2b59e00b7739a5df7f8
@fanquake
Copy link
Member

fanquake commented Nov 1, 2022

This has been merged.

@fanquake fanquake closed this Nov 1, 2022
@theStack theStack deleted the 202211-test-remove_unused_change_xprv_xpub_constants branch November 1, 2022 17:27
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Nov 1, 2022
f1ee974 test: remove unused `CHANGE_{XPRV,XPUB}` constants (Sebastian Falbesoner)

Pull request description:

  These constants exist since the introduction of the functional test wallet_taproot.py (2667366), but they have never been used.

ACKs for top commit:
  instagibbs:
    ACK f1ee974
  brunoerg:
    ACK f1ee974

Tree-SHA512: 2c6f8cd537c35af59c4cbe6abb07386b7fb9e4662c7bed27ad8935e004f9bf452705fb1caef4f5ea1f4ad509745afba966005f95a18cf2b59e00b7739a5df7f8
@bitcoin bitcoin locked and limited conversation to collaborators Nov 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants