Skip to content

Short-term fix for needProofBlock method#386

Merged
adizere merged 3 commits intov0.34.xfrom
adi/quick_fix_370
Feb 24, 2023
Merged

Short-term fix for needProofBlock method#386
adizere merged 3 commits intov0.34.xfrom
adi/quick_fix_370

Conversation

@adizere
Copy link
Contributor

@adizere adizere commented Feb 24, 2023

Closes: #402


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@adizere adizere requested a review from a team as a code owner February 24, 2023 15:00
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

Thanks!

@sergio-mena sergio-mena changed the base branch from v0.34.x to main February 24, 2023 17:50
@sergio-mena sergio-mena requested a review from a team as a code owner February 24, 2023 17:50
@sergio-mena sergio-mena changed the base branch from main to v0.34.x February 24, 2023 17:51
@adizere adizere changed the base branch from v0.34.x to main February 24, 2023 17:51
@adizere adizere changed the base branch from main to v0.34.x February 24, 2023 18:01
@adizere adizere merged commit c7dfb8c into v0.34.x Feb 24, 2023
@adizere adizere deleted the adi/quick_fix_370 branch February 24, 2023 18:37
@adizere adizere linked an issue Feb 27, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Short-term fix: Render the consensus needProofBlock method more robust

3 participants