-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Rename --gen-txs to --with-txs #2190
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2190 +/- ##
========================================
Coverage 63.62% 63.62%
========================================
Files 136 136
Lines 8270 8270
========================================
Hits 5262 5262
Misses 2652 2652
Partials 356 356 |
CHANGELOG.md
Outdated
@@ -555,7 +555,7 @@ FEATURES: | |||
* gaiad init overhaul | |||
* Create genesis transactions with `gaiad init gen-tx` | |||
* New genesis account keys are automatically added to the client keybase (introduce `--client-home` flag) | |||
* Initialize with genesis txs using `--gen-txs` flag | |||
* Initialize with genesis txs using `--with-txs` flag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think we should modify an old changelog entry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove changes to CHANGELOG
, but otherwise, utACK 👍
CHANGELOG.md
Outdated
@@ -555,7 +555,7 @@ FEATURES: | |||
* gaiad init overhaul | |||
* Create genesis transactions with `gaiad init gen-tx` | |||
* New genesis account keys are automatically added to the client keybase (introduce `--client-home` flag) | |||
* Initialize with genesis txs using `--gen-txs` flag | |||
* Initialize with genesis txs using `--with-txs` flag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added entries in
PENDING.md
with issue #rereviewed
Files changed
in the github PR explorerFor Admin Use: