Open Olshansk opened 1 year ago
Patch coverage: 2.04
% and project coverage change: -0.21
:warning:
Comparison is base (
2074a1b
) 31.52% compared to head (e9fd550
) 31.32%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
@bryanchriswhite This is still a WIP but would appreciate if you can spend some time reviewing and providing feedback, or at least ideas for future improvements. Would be nice to have one other person have some context into how this works and will evolve. Please note that my goal with this PR is to get some sort of state sync into main to unblock other features and we will then improve it. consensus/e2e_tests/state_sync_test.go
might be a good start and please ignore the commit history as it will only add confusion rather than help.
@bryanchriswhite Just FYI that I have paused work on this PR and am working on the E2E test in #829.
PTAL at this when you have a chance
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
Our GitHub checks need improvements? Share your feedbacks!
@bryanchriswhite I'm going to keep the WIP tag and keep working on it in #874 but have moved it into the ready-for-review stage in order to get more 👀 on it
@red-0ne Will look into the broken tests tomorrow.
Description
Summary generated by Reviewpad on 07 Aug 23 18:22 UTC
This pull request includes changes in multiple files. Here is a summary of the modifications:
Output Address
design.Please review these changes and let me know if you need further assistance.
Issue
Fixes #579
Type of change
Please mark the relevant option(s):
List of changes
What is this PR in its current form?
What is this PR NOT intended to be
Blockers / further improvements needed in this PR
Testing
make develop_test
; if any code changes were mademake test_e2e
on k8s LocalNet; if any code changes were madee2e-devnet-test
passes tests on DevNet; if any code was changedRequired Checklist
godoc
format comments on touched members (see: tip.golang.org/doc/comment)If Applicable Checklist
shared/docs/*
if I updatedshared/*
README(s)