Skip to content

Short-term fix for needProofBlock problem based on the main branch#387

Merged
adizere merged 3 commits intomainfrom
adi/quick_fix_370_main
Feb 27, 2023
Merged

Short-term fix for needProofBlock problem based on the main branch#387
adizere merged 3 commits intomainfrom
adi/quick_fix_370_main

Conversation

@adizere
Copy link
Contributor

@adizere adizere commented Feb 24, 2023

Ports the changes from PR #386 based off the main branch. To be backported to v0.37.*.

Ref: #402 and #370


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 18:43
@adizere adizere changed the title Adi/quick fix 370 main Short-term fix for needProofBlock problem based on the main branch Feb 24, 2023
@adizere adizere merged commit 28b130f into main Feb 27, 2023
@adizere adizere deleted the adi/quick_fix_370_main branch February 27, 2023 09:39
mergify bot pushed a commit that referenced this pull request Feb 27, 2023
#387)

* changelog

* Added log and short-circuit in needProofBlock

(cherry picked from commit 28b130f)
adizere added a commit that referenced this pull request Feb 28, 2023
#387) (#400)

* changelog

* Added log and short-circuit in needProofBlock

(cherry picked from commit 28b130f)

Co-authored-by: Adi Seredinschi <adizere@gmail.com>
Co-authored-by: Lasaro <lasaro@informal.systems>
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.

4 participants