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

Using JsonCreator cause generating invalid path reference in JsonMappingException #801

Merged
merged 3 commits into from
May 22, 2015

Conversation

Kamil-Benedykcinski
Copy link
Contributor

No description provided.

@cowtowncoder
Copy link
Member

Looks good to me!

One process thing before I can merge this: before the first contribution, we need to get a Contributor License Agreement. It's a one-page document:

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

and just needs to be filled with info on contributor, project (Jackson) and signed.
Easiest way is usually to print, scan, email to info at fasterxml dot com.

Once we have that I can merge this in, and CLA is valid for any and all contributions you make for all Jackson projects. It's a bit of pain but fortunately the only thing we need to track to keep corporate users of the library happy. :)

So, if you can do that when you get a chance, that would be great.
Thank you again for the contribution!

@Kamil-Benedykcinski
Copy link
Contributor Author

CLA has been sent:)

@Kamil-Benedykcinski
Copy link
Contributor Author

When will be next version released?

@cowtowncoder
Copy link
Member

Ah, did not notice there was separate PR for 2.5. Since this should be safe, will merge this as well.

As to next version: 2.6.0-rc2 within a week or two (and final 2.6.0 within 2-3 weeks).
Probably similarly for 2.5.4.

cowtowncoder added a commit that referenced this pull request May 22, 2015
Using JsonCreator cause generating invalid path reference in JsonMappingException
@cowtowncoder cowtowncoder merged commit 5f09320 into FasterXML:2.5 May 22, 2015
cowtowncoder added a commit that referenced this pull request May 22, 2015
@Kamil-Benedykcinski
Copy link
Contributor Author

Thanks!

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