-
Notifications
You must be signed in to change notification settings - Fork 221
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 for BorshDeserialize and BorshSerialize #640
Comments
Hey @ivanceras. If there's enough demand for it, we might consider adding this feature. However, that would mean tying any major releases to a breaking change to ethereum-types, which we'd like to avoid. |
Borsh support would be useful to hyperlane and eclipse. @ordian to address your concern about avoiding being tied to breaking changes - given that |
@sloboste hey, sorry for the long delay in reply. I think your suggestion is a reasonable workaround we can live with. |
@tkporter yes, see point 2 in #698 (comment). Instead of introducing a new crate with a regular macro, we could just use derive with a feature gate. |
Hi, is there any plan to support BorshSerialize and BorshDeserialize for the ethereum-types.
I notice that borsh is used in NEAR and in Solana which provides compatibility with ethereum.
The text was updated successfully, but these errors were encountered: