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
ABI types for utils #61
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.
It seems like it would be helpful to list the fields and their types in the docs for reference. Otherwise the user has to go to code too. What you think?
Edit: the docs already have autodoc
in there, so you could add the definition as a docstring and have it construct the docs for you.
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!
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 was wrong?
Related to Issue #
Closes #
How was it fixed?
Types for elements of a contract ABI. These were lifted from web3.py to be usable in
eth-utils
.Updates in web3.py and eth-utils inbound.
Todo:
Clean up commit history
Add or update documentation related to these changes
Add entry to the release notes
Cute Animal Picture