Skip to content
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

fix: line length violations in abci.proto and CI environment notes #13

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 5 additions & 6 deletions packages/valory/protocols/abci/abci.proto
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,11 @@ message AbciMessage{

// Custom Types
message CheckTxType{
enum CheckTxTypeEnum {
enum CheckTxType {
NEW = 0;
RECHECK = 1;
}
CheckTxTypeEnum type = 1;
CheckTxType type = 1;
}

message ConsensusParams{
Expand Down Expand Up @@ -66,8 +66,8 @@ message AbciMessage{
int64 height = 3;
// The corresponding time where the offense occurred
Timestamp time = 4;
// Total voting power of the validator set in case the ABCI application
// does not store historical validators.
// Total voting power of validator set if app lacks
// historical validator data.
// https://github.com/tendermint/tendermint/issues/4581
int64 total_voting_power = 5;
}
Expand Down Expand Up @@ -104,8 +104,7 @@ message AbciMessage{
bytes next_validators_hash = 9; // validators for the next block
bytes consensus_hash = 10; // consensus params for current block
bytes app_hash = 11; // state after txs from the previous block
// root hash of all results from previous block transactions
bytes last_results_hash = 12;
bytes last_results_hash = 12; // root hash of previous block results

// consensus info
bytes evidence_hash = 13; // evidence included in the block
Expand Down