-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Added keccak256 tests for Hasher #5172
Conversation
The Check Links test failed due to the site docs.rs going down during the check. It needs to be run again. Also, I don't have permissions to add assignees, reviewers, or labels. |
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 great, thanks for contributing!
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 haven't verified that the resulting hashes are correct so I'm trusting the implementation to produce the correct hashes
Co-authored-by: Braqzen <103777923+Braqzen@users.noreply.github.com>
Co-authored-by: Braqzen <103777923+Braqzen@users.noreply.github.com>
Description
Closes #5090
Checklist
Breaking*
orNew Feature
labels where relevant.