-
Notifications
You must be signed in to change notification settings - Fork 16
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
chore: bump to latest version #28
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.
I looked over the reference changes and made a few suggestions for spelling (ie. Merkle over merkle, since it's a name). The definitions look correct and straight forward to me. You may want to get additional eyes from others on the Client team if possible. I did not check the correctness of the examples by actually running them - are these changes confirmed? Overall, lgtm.
Co-authored-by: Brandon Vrooman <brandon.vrooman@gmail.com>
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.
I agree with @bvrooman here, the doc changes look good, but I didn't try out all of the examples either. Assuming those work, LGTM.
This PR bumps the docs to reflect the latest version of fuel-core