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.

Sign up for GitHub

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

Jump to bottom

Serialization refactor: macros for the new traits #802

Closed
Tracked by #463
piodul opened this issue Aug 25, 2023 · 2 comments
Closed
Tracked by #463

Serialization refactor: macros for the new traits #802

piodul opened this issue Aug 25, 2023 · 2 comments
Assignees
@piodul
Labels
area/proc-macros Related to procedural macros
Milestone
0.11.0

Comments

@piodul
Copy link
Collaborator

piodul commented Aug 25, 2023

Sub-task of #463.

The new SerializeRow and SerializeCql traits will need to have their corresponding procedural macros implemented. Those macros should match struct fields to columns/UDT fields by name. This should be done by default, but there should be an attribute that allows disabling it.
@piodul piodul mentioned this issue Aug 25, 2023
Closed
@piodul piodul self-assigned this Aug 25, 2023
@piodul
Copy link
Collaborator Author

piodul commented Aug 25, 2023

This depends on the definition of the traits in #801.

@piodul piodul mentioned this issue Aug 25, 2023
Closed
@piodul piodul added this to the 0.10.0 milestone Aug 25, 2023
@piodul piodul mentioned this issue Oct 20, 2023
Merged
8 tasks
@piodul piodul added the area/proc-macros Related to procedural macros label Oct 26, 2023
@piodul piodul mentioned this issue Dec 15, 2023
Merged
8 tasks
@Lorak-mmk
Copy link
Collaborator

This was implemented in #851 with some smaller follow ups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@piodul piodul

Labels
area/proc-macros Related to procedural macros
Projects
None yet
Milestone
0.11.0
Development

No branches or pull requests
2 participants
@piodul @Lorak-mmk