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

Fix deserialization codegen with manually implemented builder. #1209

Merged

Conversation

davidmorgan
Copy link
Collaborator

No description provided.

@davidmorgan davidmorgan force-pushed the fix-manual-builder-deserialization branch 4 times, most recently from c724174 to b650c21 Compare January 11, 2023 11:16
@iinozemtsev
Copy link
Contributor

LG

@davidmorgan davidmorgan force-pushed the fix-manual-builder-deserialization branch 2 times, most recently from f270b92 to 37d94fb Compare January 11, 2023 13:09
The generated code now checks at runtime if the builder method auto-instantiated the nested builder.

Add `FieldMixin` to reuse code between value generator and serializer generator.
@davidmorgan davidmorgan force-pushed the fix-manual-builder-deserialization branch from 37d94fb to 7613eb6 Compare January 11, 2023 13:10
@davidmorgan davidmorgan merged commit 316ad1d into google:master Jan 11, 2023
@davidmorgan davidmorgan deleted the fix-manual-builder-deserialization branch January 11, 2023 13:15
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.

None yet

2 participants