-
Notifications
You must be signed in to change notification settings - Fork 151
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
SSZ implementation for alloy primitives #407
Conversation
Do you mind sharing more about your usecase? Are there other primitive types that would be useful with SSZ? We have so far restricted alloy to EL-layer tooling. |
Hi @prestwich, we are running a custom reth implementation and we had to implement SSZ traits for alloy types. So we thought that it might be useful to share our implementation and just consume it from alloy-primitives. Other 2 types that we also implemented are U64 and U256 from |
Cool, let me sync with the other maintainers and get back to this Separately, I'd be in favor of a PR for SSZ support in the ruint support modules |
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.
Fine with me
Sure will also take care of it. |
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!
Thank you! |
Motivation
I found my self needing to wrap alloy primitives types and implement SSZ on it. I thought it could be useful to have those trait implementations downstream.
Solution
Implementing SSZ as a conditional flag for alloy primitives.
PR Checklist