-
Notifications
You must be signed in to change notification settings - Fork 18
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 @JsonDeserialize(builder=..) #527
Conversation
what exactly are the pre-instantiate hooks used for (theyre used in data right)? i think builders should have the same behavior as creator methods wrt pre-instantiate hooks. i think atm the hook is called after the creator but before any more properties are set. |
I am not sure about these pre-instantiate hooks myself, @dstepanov will have to confirm |
The hook is used to perform possible validation before the constructor is invoked. In this case, we need to think about how we would trigger the builder method validation. |
SonarCloud Quality Gate failed. 0 Bugs 54.9% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Draft PR for builder support, some open questions: