Closed h5law closed 1 year ago
Patch coverage: 7.22
% and project coverage change: -0.06
:warning:
Comparison is base (
e64af30
) 30.86% compared to head (d4e1694
) 30.80%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
@dylanlott I think this one is close to approving but figured we should probably merge persistence/tree-store
to main first and rebase/merge this with main?
Lmk if you have a preference
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!
Description
Summary generated by Reviewpad on 23 Jun 23 10:35 UTC
This pull request contains several file diffs with various updates and additions. The implementation of the
setupTrees
function inmodule.go
has been updated to use a newstateTree
struct and additional error handling. ThePocket V1
protocol state hash document has been updated with changes to various infrastructural components and the computation of the state hash. Thepersistence_module.go
file has a newGetTreeStore()
method, andtreestore_module.go
file has a new import statement andGetTree()
function. Additionally, theutility/session_test.go
andstate.go
test cases have been updated with new values and logic changes.Issue
Fixes #834
Type of change
Please mark the relevant option(s):
List of changes
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)