Skip to content

abci++: plumb vote extension methods into interface#9532

Closed
cmwaters wants to merge 46 commits intofeature/abci++veffrom
cal/vote-extensions-1
Closed

abci++: plumb vote extension methods into interface#9532
cmwaters wants to merge 46 commits intofeature/abci++veffrom
cal/vote-extensions-1

Conversation

@cmwaters
Copy link
Contributor

This PR does all the initial plumage, adding the required methods to the interface without actually calling them in any situations.

This is predominantly from #8031 but also refers to prior work #6646 and #6885

NOTE: This builds on top of the finalize-block work

@cmwaters cmwaters requested a review from a team October 10, 2022 10:30
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale for use by stalebot label Oct 21, 2022
@thanethomson thanethomson removed the stale for use by stalebot label Oct 21, 2022
@cmwaters cmwaters mentioned this pull request Oct 24, 2022
3 tasks
@cmwaters cmwaters self-assigned this Oct 26, 2022
@faustbrian
Copy link

@cmwaters could you rebase this with cal/finalize-block so that your protobuf changes from that PR are reflected here?

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale for use by stalebot label Nov 9, 2022
@sergio-mena sergio-mena removed the stale for use by stalebot label Nov 9, 2022
@cmwaters cmwaters marked this pull request as ready for review November 17, 2022 14:52
@thanethomson thanethomson added the C:abci Component: Application Blockchain Interface label Nov 28, 2022
Base automatically changed from cal/finalize-block to feature/abci++vef November 28, 2022 22:12
@sergio-mena sergio-mena assigned sergio-mena and unassigned cmwaters Nov 29, 2022
@sergio-mena
Copy link
Contributor

Closing this PR to consolidate it with #9620

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:abci Component: Application Blockchain Interface

Projects

Status: Done/Merged

Development

Successfully merging this pull request may close these issues.

4 participants