-
Notifications
You must be signed in to change notification settings - Fork 0
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
editorial pass #1
editorial pass #1
Conversation
Signed-off-by: bumblefudge <[email protected]>
Signed-off-by: bumblefudge <[email protected]>
@zachferland i can't tag you for review but you should probably check nothign I rephrased got mangled, particularly this and this |
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.
thanks @bumblefudge, minor wording change to your edits, let me know if that makes sense
32 byte argument is the merkle-root CID. A CID is more than 32 bytes, and a | ||
partial CID is used to allow the argument to be efficiently packed in the | ||
transaction. Function name or contract does not matter, and is up to the | ||
implementation. Use of a root CID encoded in [DAG-CBOR][] is recommended. |
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.
👍
Co-authored-by: Zach Ferland <[email protected]>
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.
thanks! lgtm
Signed-off-by: bumblefudge [email protected]