Skip to content

refactor: provide a current block commit with request finalize block request#501

Merged
shotonoff merged 17 commits intov0.10-devfrom
td-128-add-commit-to-finalize-block-request
Nov 8, 2022
Merged

refactor: provide a current block commit with request finalize block request#501
shotonoff merged 17 commits intov0.10-devfrom
td-128-add-commit-to-finalize-block-request

Conversation

@shotonoff
Copy link
Collaborator

@shotonoff shotonoff commented Nov 7, 2022

Issue being fixed or feature implemented

Tenderdash should provide a block commit during the block finalization

What was done?

Replaced a last commit (RequestFinalizeBlock.DecidedLastCommit) on a current block commit (RequestFinalizeBlock.Commit)

How Has This Been Tested?

Tests

Breaking Changes

N/A

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

Copy link
Collaborator

@lklimek lklimek left a comment

Choose a reason for hiding this comment

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

LGTM (after #498 )

@shotonoff shotonoff merged commit 78fd435 into v0.10-dev Nov 8, 2022
@shotonoff shotonoff deleted the td-128-add-commit-to-finalize-block-request branch November 8, 2022 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants