-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
Add a way to distinguish between null and empty #471
Add a way to distinguish between null and empty #471
Conversation
Since this is API addition, SemVer would not allow us to backport it. In practical terms, it is unlikely new 2.15 versions will be released. But I could perhaps be convinced that merging into 2.17 (for 2.17.1) would be ok as "dark launch" (that is, officially feature is only available on 2.18 but in practice is included earlier if you know where to look). |
That would be great to have it on 2.17.1 After that, what will be the next step? |
I can cherry-pick change back to 2.17 if everything works, so 2.18 is fine. But one other thing I need is CLA, from: https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf and the usual way is to print it, fill & sign, scan/photo, email to I hope to review this PR soon, hopefully today. |
Document sent, Thanks |
All good with CLA; I have been busy but will try to get this reviewed and merged tomorrow. |
Ok, so: this went officially in 2.18 for 2.18.0, but I did cherry-pick it for 2.17 branch: so technically it will go in as undocumented feature of 2.17.1 ("dark launch" so to speak). |
Great thanks a lot :) |
Following the issue: #469
I made a pull request with that proposal:
Note that I made this PR on the branch 2.18, but I'm using the version 2.15 in my projects (Spring boot 3.1)
If accepted we can move this feature also to previous versions