-
Notifications
You must be signed in to change notification settings - Fork 337
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
Fix result of debug_traceBlockByNumber #3060
Conversation
WASM runtime size check:Compared to target branchMoonbase runtime: 2248 KB (no changes) ✅ Moonbeam runtime: 2228 KB (no changes) ✅ Moonriver runtime: 2224 KB (no changes) ✅ Compared to latest release (runtime-3300)Moonbase runtime: 2248 KB (+220 KB compared to latest release) Moonbeam runtime: 2228 KB (+232 KB compared to latest release) Moonriver runtime: 2224 KB (+232 KB compared to latest release) |
Coverage Report@@ Coverage Diff @@
## master rq/add-tx-hash +/- ##
==================================================
- Coverage 74.74% 74.72% -0.02%
Files 369 369
+ Lines 94269 94295 +26
==================================================
Hits 70453 70453
+ Misses 23816 23842 +26
|
I made some changes! but it's roughly what we spoke, looks good :) |
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
What does it do?
Things missing @gonzamontiel:
Example request:
Actual result:
Expected result example: