-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: add missing Tx receipt info #1931
feat: add missing Tx receipt info #1931
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #1931 +/- ##
==========================================
+ Coverage 72.93% 73.07% +0.14%
==========================================
Files 417 425 +8
Lines 51376 52131 +755
==========================================
+ Hits 37471 38096 +625
- Misses 13905 14035 +130
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 61 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
good progress, left some suggestions and pointers.
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.
a few additional suggestions, but this is pretty close.
ref EIP-98 I think status root has been replaced with status code (ref EIP-658) which we already have in TransactionReceipt |
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.
getting there.
left more suggestions and requests
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.
nice work!
Closes #1922
TODO:
Remaining log fields :