Repository navigation
chore(beacon): drand protobuf and drand fake signer - #7747
EclesioMeloJunior wants to merge 2 commits into
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
WalkthroughThe changes add a test-only protobuf response type and a fake drand signer. A test encodes and decodes entries for rounds 1–5, then checks that the beacon verifies them. ChangesDrand test support
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to
|
Codecov Report❌ Patch coverage is
Additional details and impacted files
... and 7 files with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
akaladarshi
left a comment
There was a problem hiding this comment.
@EclesioMeloJunior the PR title is not actually feat it should either test or chore, since changes are not adding any features.
Pull request was converted to draft
67000a8 to
a129a48
Compare
Summary of changes
drandgossipsub proto definitionChanges introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
Outside contributions
Summary by CodeRabbit