-
Notifications
You must be signed in to change notification settings - Fork 102
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
Return full gas report to wasmd #448
Conversation
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.
Looks good. We also need to update the caller code (Go) of the changed functions in internal/api
75c0a5c
to
771768a
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.
Looks good! We might need to use this PR branch for the reproduction of the issue though due to the API breaking change.
e6f5f27
to
c1259c1
Compare
c1259c1
to
0d4dbb6
Compare
0d4dbb6
to
def004f
Compare
Closing this, as we don't need to return all this information to wasmd |
This adjusts the entry points to take a pointer to the full gas report.
I'll integrate it into the go side when this is merged and the new library is built.