Skip to content

Divergences in comparison with #9620. Part 1: easy/obvious#9888

Merged
sergio-mena merged 2 commits intofeature/abci++veffrom
sergio/9620-divergences
Dec 16, 2022
Merged

Divergences in comparison with #9620. Part 1: easy/obvious#9888
sergio-mena merged 2 commits intofeature/abci++veffrom
sergio/9620-divergences

Conversation

@sergio-mena
Copy link
Contributor

@sergio-mena sergio-mena commented Dec 15, 2022

Contributes to #9887

These are the obvious divergences (including bugs) between #9620 and feature/abci++vef

For more info, check #9887


PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

@sergio-mena sergio-mena requested a review from a team December 15, 2022 21:31
bytes extension_signature = 10;
}

// VoteExtension is app-defined additional information to the validator votes.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These left-overs may have caused a breakage in the block format

Co-authored-by: Thane Thomson <connect@thanethomson.com>
@sergio-mena sergio-mena merged commit a749afe into feature/abci++vef Dec 16, 2022
@sergio-mena sergio-mena deleted the sergio/9620-divergences branch December 16, 2022 11:16
@sergio-mena sergio-mena self-assigned this Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done/Merged

Development

Successfully merging this pull request may close these issues.

2 participants