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

abci-cli: PrepareProposal and ProcessProposal #9281

Merged
merged 4 commits into from
Aug 17, 2022

Conversation

sergio-mena
Copy link
Contributor

Closes #9270

The changes to CHANGELOG_PENDING.md are the ones cherrypicked.
There will be an overall review of CHANGELOG_PENDING.md regarding ABCI++ when merging back to main


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

jmalicevic and others added 2 commits August 17, 2022 11:18
…ci-cli (#8901)

* Add `process_proposal` command to abci-cli

* Added process proposal to the 'tutorial' examples

* Added entry in CHANGELOG_PENDING.md

* Allow empty blocks in PrepareProposal, ProcessProposal, and FinalizeBlock

* Fix minimum arguments

* Add tests for empty block

* Updated abci-cli doc

Co-authored-by: Sergio Mena <sergio@informal.systems>
Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com>
@sergio-mena sergio-mena requested a review from ebuchman as a code owner August 17, 2022 14:52
@sergio-mena sergio-mena requested a review from a team August 17, 2022 14:52
@sergio-mena sergio-mena self-assigned this Aug 17, 2022
abci/tests/test_cli/ex1.abci Outdated Show resolved Hide resolved
@sergio-mena sergio-mena changed the title Sergio/9270 abci cli abci-cli: PrepareProposal and ProcessProposal Aug 17, 2022
@sergio-mena sergio-mena merged commit 622b930 into feature/abci++ppp Aug 17, 2022
@sergio-mena sergio-mena deleted the sergio/9270-abci-cli branch August 17, 2022 18:47
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.

4 participants