-
Notifications
You must be signed in to change notification settings - Fork 253
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
feat!: specify utia
as denom for gov module genesis
#1484
Conversation
// HACKHACK explicitly ignore the error returned from json.Unmarshal because | ||
// the error is a failure to unmarshal the string StartingProposalId as a | ||
// uint which is unrelated to the test here. | ||
//nolint:errcheck | ||
json.Unmarshal(raw, &govGenesisState) |
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.
[optional]
this comment is good but also why not explicitly ignore the error and get rid of the no lint?
// HACKHACK explicitly ignore the error returned from json.Unmarshal because | |
// the error is a failure to unmarshal the string StartingProposalId as a | |
// uint which is unrelated to the test here. | |
//nolint:errcheck | |
json.Unmarshal(raw, &govGenesisState) | |
// HACKHACK explicitly ignore the error returned from json.Unmarshal because | |
// the error is a failure to unmarshal the string StartingProposalId as a | |
// uint which is unrelated to the test here. | |
_ = json.Unmarshal(raw, &govGenesisState) |
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.
we just need to get rid of the :
in the first image since we're not declaring anything
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.
oof how did I miss that. Can do in a follow-up PR.
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.
nah who care lol
Addresses this feedback: #1484 (comment)
Fixes an issue identified in https://github.com/celestiaorg/itn-doc/issues/168
Testing
Verified genesis.json contains
utia
in the gov module in this diff