-
-
Notifications
You must be signed in to change notification settings - Fork 343
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: revert back to previous getPendingBalanceToWithdraw
implementation
#7419
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #7419 +/- ##
=========================================
Coverage 50.26% 50.26%
=========================================
Files 602 602
Lines 40376 40375 -1
Branches 2206 2206
=========================================
Hits 20293 20293
+ Misses 20043 20042 -1
Partials 40 40 |
Performance Report✔️ no performance regression detected Full benchmark results
|
@nflaig ideally we should have unit test if not spec test to reproduce/fix the issue |
we've added a spec test that covers this scenario and it's already part of |
🎉 This PR is included in v1.27.0 🎉 |
Since ChainSafe/ssz#456 it's possible to use
getAllReadonly()
with uncommited changes. This PR essential reverts changes done in #7375 as it causes more memory allocation which is not ideal.