Skip to content

cli: Add option to not clear address book with unsafe reset#3606

Merged
melekes merged 2 commits intotendermint:developfrom
climber73:issues/3585
May 1, 2019
Merged

cli: Add option to not clear address book with unsafe reset#3606
melekes merged 2 commits intotendermint:developfrom
climber73:issues/3585

Conversation

@climber73
Copy link
Contributor

@climber73 climber73 commented Apr 30, 2019

Adds --keep-addr-book flag to unsafe_reset_all command allowing not to clear address book

@climber73
Copy link
Contributor Author

Not sure, maybe it's better to split the definitions of ResetAllCmd and ResetPrivValidatorCmd

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (develop@40dbad9). Click here to learn what that means.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             develop    #3606   +/-   ##
==========================================
  Coverage           ?   64.29%           
==========================================
  Files              ?      213           
  Lines              ?    17449           
  Branches           ?        0           
==========================================
  Hits               ?    11218           
  Misses             ?     5319           
  Partials           ?      912
Impacted Files Coverage Δ
cmd/tendermint/commands/reset_priv_validator.go 5.88% <33.33%> (ø)

Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

🥑

@melekes
Copy link
Contributor

melekes commented Apr 30, 2019

Thanks for contributing to Tendermint 👍

Copy link
Contributor

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

🚀

@liamsi
Copy link
Contributor

liamsi commented Apr 30, 2019

Awesome! Thanks ❤️ Even if the code is self-explanatory: Could you add a short description on what was changed and why in the PR description?

@melekes melekes merged commit a2a68df into tendermint:develop May 1, 2019
@climber73 climber73 deleted the issues/3585 branch May 1, 2019 06:16
@melekes melekes mentioned this pull request May 7, 2019
36 tasks
@melekes melekes mentioned this pull request May 30, 2019
44 tasks
brapse pushed a commit to brapse/tendermint that referenced this pull request Jun 5, 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.

4 participants