Skip to content
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

feat: abi-json crate #81

Merged
merged 2 commits into from
Jun 8, 2023
Merged

feat: abi-json crate #81

merged 2 commits into from
Jun 8, 2023

Conversation

DaniPopes
Copy link
Member

@DaniPopes DaniPopes commented Jun 8, 2023

@prestwich

Use a macro to declare 2 structs for each item to be able to derive serde.
Use the derived internally tagged enum impl in both the public enum, and the public structs

@DaniPopes DaniPopes requested a review from prestwich June 8, 2023 19:08
@DaniPopes DaniPopes changed the base branch from prestwich/abi-json to main June 8, 2023 19:17
@DaniPopes DaniPopes changed the title refactor: derive Serde impls, use internally tagged enum feat: abi-json crate Jun 8, 2023
@prestwich prestwich changed the base branch from main to prestwich/abi-json June 8, 2023 19:28
@prestwich prestwich merged commit a129506 into prestwich/abi-json Jun 8, 2023
@DaniPopes DaniPopes deleted the dani/abi-json branch June 21, 2023 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants