-
Notifications
You must be signed in to change notification settings - Fork 160
chore: cherry pick #13479 - redirectForAccount proxy contract support #13668
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
Conversation
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
…re clear Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
…arify where the function selector bytes comes from Signed-off-by: lukelee-sl <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/0.51 #13668 +/- ##
===================================================
+ Coverage 0 67.57% +67.57%
- Complexity 0 36389 +36389
===================================================
Files 0 3717 +3717
Lines 0 149480 +149480
Branches 0 15557 +15557
===================================================
+ Hits 0 101011 +101011
- Misses 0 44100 +44100
- Partials 0 4369 +4369 ☔ View full report in Codecov by Sentry. |
Node: HAPI Test (Token) Results 20 files 20 suites 6m 6s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Restart) Results3 tests 3 ✅ 6m 12s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Node Death Reconnect) Results3 tests 3 ✅ 6m 17s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Smart Contract) Results 68 files 68 suites 23m 45s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Time Consuming) Results19 tests 19 ✅ 21m 36s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Misc) Results 51 files 51 suites 23m 36s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Crypto) Results 23 files 23 suites 12m 17s ⏱️ Results for commit 8645fb4. ♻️ This comment has been updated with latest results. |
Signed-off-by: lukelee-sl <[email protected]>
Signed-off-by: lukelee-sl <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And includes "[feat: remove max gas pure check]" b2294fa
#13648
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tyvm @lukelee-sl !
Description:
Cherry pick #13479. This cherry-pick does contain the same number of files in the original pr (#13479). The reason is that this pr was based off of pr #13326 which was separately checked in. The commit history for this pr starts with commit a8a4855 and all subsequent commits (minus the Merge branch develop commits) were included in this cherry pick.