remove serde support from reth-codec

paradigmxyz / reth

Modular, contributor-friendly and blazing-fast implementation of the Ethereum protocol, in Rust

https://reth.rs/

Apache License 2.0

3.97k stars 1.2k forks source link

remove serde support from reth-codec #9558

Closed mattsse closed 4 months ago

mattsse commented 4 months ago

Describe the feature

the reth-codec macro is a bit bloated

https://github.com/paradigmxyz/reth/blob/d5964cae0e18a509e5ef1b08fa094d7278ebd4d1/crates/storage/codecs/derive/src/lib.rs#L48-L54

this does not belong in there.

TODO

remove serde auto gen from doce macro and add support natively

Additional context

No response

EmmanuelAR commented 4 months ago

Hello @mattsse

My name is Emmanuel, a software developer from Costa Rica. I would be happy to assist with this task. I will review the current implementation and could remove serde auto gen from doce macro and add support natively

Please let me know if this is acceptable.

Best regards, Emmanuel 😁