-
Notifications
You must be signed in to change notification settings - Fork 1.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
Support skip
attribute in RlpEncodable derive macro
#1102
Comments
I would like to take this on |
I guess we'd need |
Yeah, I found this: reth/crates/common/rlp-derive/src/lib.rs Lines 12 to 16 in d86c989
but looks like it's not implemented at all. Do we just need |
yeh I believe this was not implemented before the fork. |
I will see how to add |
Describe the feature
There are use cases where being able to skip fields during rlp encoding is useful:
#1101
Currently, we don't support this in the
RlpEncodable
derive macro.TODO
add
rlp(skip)
field attribute support forRlpEncodable
macro that should work the same asserde(skip)
.Additional context
No response
The text was updated successfully, but these errors were encountered: