-
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
Proto encoding of gov and related modules #5493
Conversation
c588458
to
36fbe5b
Compare
This pull request introduces 3 alerts when merging 347e041 into 2140244 - view on LGTM.com new alerts:
|
484ea2f
to
fe8fa35
Compare
Codecov Report
@@ Coverage Diff @@
## master #5493 +/- ##
===========================================
+ Coverage 35.46% 48.03% +12.56%
===========================================
Files 331 315 -16
Lines 32612 21599 -11013
===========================================
- Hits 11567 10375 -1192
+ Misses 19819 10425 -9394
+ Partials 1226 799 -427
|
fe8fa35
to
372dff8
Compare
@@ -1,6 +1,7 @@ | |||
package keeper | |||
|
|||
import ( | |||
proto "github.com/gogo/protobuf/types" |
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.
File is not goimports
-ed (from goimports
)
proto "github.com/gogo/protobuf/types" |
@@ -45,7 +46,8 @@ func TestProposalQueues(t *testing.T) { | |||
|
|||
activeIterator := keeper.ActiveProposalQueueIterator(ctx, proposal.VotingEndTime) | |||
require.True(t, activeIterator.Valid()) | |||
keeper.cdc.UnmarshalBinaryLengthPrefixed(activeIterator.Value(), &proposalID) | |||
require.Equal(t, proposalID, proposal.ProposalID) | |||
var proposalIdWrapper proto.UInt64Value |
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.
var proposalIdWrapper
should be proposalIDWrapper
(from golint
)
087b788
to
5faad87
Compare
…rything besides proposal)
Co-Authored-By: Bot from GolangCI <42910462+golangcibot@users.noreply.github.com>
97aef12
to
8bd648e
Compare
Description
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)