Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accounts V2: Add --num-accounts to HD wallet commands #6721

Merged
merged 17 commits into from
Jul 27, 2020
Merged

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented Jul 25, 2020

What type of PR is this?
Feature

What does this PR do? Why is it needed?
Part of #6220 , this PR adds a flag --num-accounts for declaring how many accounts to be generated with accounts-v2 create for HD wallets. It also lets you define how many accounts to recover with wallet-v2 recover when recovering a mnemonic phrase is complete.

@0xKiwi 0xKiwi requested a review from a team as a code owner July 25, 2020 05:19
@codecov
Copy link

codecov bot commented Jul 26, 2020

Codecov Report

Merging #6721 into master will increase coverage by 2.13%.
The diff coverage is 73.31%.

@@            Coverage Diff             @@
##           master    #6721      +/-   ##
==========================================
+ Coverage   60.07%   62.20%   +2.13%     
==========================================
  Files         323      384      +61     
  Lines       27422    29519    +2097     
==========================================
+ Hits        16473    18363    +1890     
+ Misses       8733     8542     -191     
- Partials     2216     2614     +398     

@0xKiwi 0xKiwi added AccountsV2 Ready For Review A pull request ready for code review labels Jul 26, 2020
@rauljordan rauljordan merged commit 7fff4ec into master Jul 27, 2020
@delete-merged-branch delete-merged-branch bot deleted the generate-n branch July 27, 2020 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants