Rename Message
struct to IbftMessage
#97
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In order to update
libp2p
dependencies inEdge
, we need to rename theMessage
struct inmessages.proto
(which carries the ibft messages likeRoundChange
,Prepare
, etc.), because,libp2p
and itspubsub
implementation introduced amessage.proto
which has the struct with the same name in it, so we get a panic over a same name panic:This PR renames the
Message
struct toIbftMessage
, so that we avoid this panic when updating thelibp2p
dependency.Changes include
Checklist