-
Notifications
You must be signed in to change notification settings - Fork 137
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 new fields to message trace for supporting new trace api #1823
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1823 +/- ##
==========================================
+ Coverage 75.21% 75.22% +0.01%
==========================================
Files 149 149
Lines 14592 14600 +8
==========================================
+ Hits 10975 10983 +8
Misses 3617 3617
|
0f08bde
to
9e16205
Compare
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.
FYI @alexytsu
a515dd2
to
8a00486
Compare
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.
One change, but then feel free to dismiss my review and merge it.
Related: #1793 and https://github.com/filecoin-project/fvm-pm/issues/613
This PR adds the following fields to
ExecutionEvent
which are required to confirm to the trace api spec:gas_limit
: The gas provided by the senderread_only
: Whether the call is read only or not