-
Notifications
You must be signed in to change notification settings - Fork 175
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
JsonNode, ArrayNode and ObjectNode extensions #489
Conversation
This seems like potentially useful (if I understand the intent) additions.
|
@cowtowncoder sure, done |
Thanks! I'll let module owners review this otherwise. This may still make it in 2.13.0 if they approve (I pushed last rc, 2.13.0-rc2 last night, but 2.13.0 won't be out until next weekend or weekend after that). |
@sokomishalov Have you signed the CLA? If not, would you fill it out and email a scan/photo of the result to info at fasterxml dot com? https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf |
@dinomite I've signed and sent the document to the mailbox, also made changes due to your suggestions, please have a look again |
CLA received just like @sokomishalov said. |
I wonder if it'd make sense to add one more cla-related labels (cla-needed, cla-available ... or something)? |
Implements #492