-
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
ensure non-nil gentxs #5497
ensure non-nil gentxs #5497
Conversation
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.
Given that this sprang from a bug report, we definitely need a test case here
added tests. |
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.
ACK, mind adding a bug fix entry on the CHANGELOG?
added changelog entry. |
fae7b4a
to
b24af20
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Are there any other blockers to this PR merging? |
CHANGELOG.md
Outdated
@@ -250,6 +250,8 @@ to detail this new feature and how state transitions occur. | |||
* (keys) Fix ledger custom coin type support bug | |||
* (x/gov) [\#5107](https://github.com/cosmos/cosmos-sdk/pull/5107) Sum validator operator's all voting power when tally votes | |||
* (rest) [\#5212](https://github.com/cosmos/cosmos-sdk/issues/5212) Fix pagination in the `/gov/proposals` handler. | |||
* (genesis) [\#5086](https://github.com/cosmos/cosmos-sdk/issues/5086) Ensure |
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.
Changes look good, however, we've released a version since this PR. So we need to move this entry up to the Unreleased
section.
Looks good @cbarraford -- sorry for the delay! All we need to do is fix the changelog and we can merge 👍 |
e004b3c
to
a5e469f
Compare
Codecov Report
@@ Coverage Diff @@
## master #5497 +/- ##
==========================================
- Coverage 44.64% 44.64% -0.01%
==========================================
Files 324 324
Lines 24707 24709 +2
==========================================
Hits 11031 11031
- Misses 12616 12618 +2
Partials 1060 1060
|
Ok, done. |
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.
ACK
Closes: #5086
Description
Ensure we never have
null
in our genesis for gentxs.For contributor use:
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerFor admin use:
WIP
,R4R
,docs
, etc)