Skip to content

Conversation

@instagibbs
Copy link
Collaborator

@instagibbs instagibbs commented Mar 4, 2019

Returns both internal an external chains when --path and --internal are not given as arguments.

@instagibbs
Copy link
Collaborator Author

@Sjors @achow101 concept ACKs?

@achow101
Copy link
Member

achow101 commented Mar 6, 2019

Concept ACK

@Sjors
Copy link
Member

Sjors commented Mar 8, 2019

Concept ACK, but do fix the tests :-)

Consider rebasing on the first commit of #137, which extracts the descriptor from the importmulti specific stuff.

@jb55 jb55 mentioned this pull request May 14, 2019
@achow101
Copy link
Member

achow101 commented Jun 16, 2019

Fix tests please? I would like to see this merged.

@instagibbs
Copy link
Collaborator Author

rebased, I have to first make sure ledger tests are passing for master then I'll fix tests

@instagibbs
Copy link
Collaborator Author

Looks like I got it working properly.

@achow101 review ready

@achow101
Copy link
Member

ACK b1b7212

@achow101 achow101 merged commit b1b7212 into bitcoin-core:master Jun 17, 2019
achow101 added a commit that referenced this pull request Jun 17, 2019
b1b7212 Create internal and external keypools by default (Gregory Sanders)

Pull request description:

  Returns both internal an external chains when `--path` and `--internal` are not given as arguments.

ACKs for commit b1b721:
  achow101:
    ACK b1b7212

Tree-SHA512: efd45b75a2def8ebb75afef2beb31806b13a26a3ca67a3b64e6e781fc64087e4cc89f4a763a1ccd685bbc8edb7bf22c95dc1b7cc59b2055a8c74f849a4978325
@Sjors Sjors mentioned this pull request Aug 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants