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

Remove usage of _neitherNull on findProperty #1918

Merged
merged 2 commits into from
Feb 5, 2018
Merged

Remove usage of _neitherNull on findProperty #1918

merged 2 commits into from
Feb 5, 2018

Conversation

deblockt
Copy link
Contributor

@deblockt deblockt commented Jan 30, 2018

Remove usage of _neitherNull on findProperty to get custom property if possible.

linked issue : #1912

This PR is done for 2.9 version

Thomas Deblock added 2 commits January 30, 2018 09:40
Remove usage of _neitherNull on findProperty to get custom property if possible.
@deblockt deblockt changed the title Fix #1912 Remove usage of _neitherNull on findProperty Jan 31, 2018
@cowtowncoder
Copy link
Member

(copied from older PR)

One other minor thing: I'm happy to merge this, but one thing we need before the first contribution to Jackson projects is CLA: (unless we already got one -- apologies if I missed earlier one, tried to check)

https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

Usual way is to print, fill, sign, scan & email toinfo at fasterxml dot com.
There is also alternative Corporate CLA if you prefer, most contributions are under individual one.

@cowtowncoder cowtowncoder merged commit 8098ff3 into FasterXML:2.9 Feb 5, 2018
@cowtowncoder cowtowncoder changed the title Remove usage of _neitherNull on findProperty Remove usage of _neitherNull on findProperty Feb 5, 2018
@deblockt
Copy link
Contributor Author

deblockt commented Feb 5, 2018

@cowtowncoder did you know when this bugfix will be released?

@cowtowncoder
Copy link
Member

@deblockt Since 2.9.4 was just released, probably takes at least 1 month more. So perhaps late march or early april. There is no time-based release schedule but more of "when there are enough fixes bundled".

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.

2 participants