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

Proposed fix for issue #703 #704

Merged
merged 1 commit into from
Feb 13, 2015
Merged

Conversation

flozano
Copy link
Contributor

@flozano flozano commented Feb 12, 2015

Proposed fix for issue #703

@cowtowncoder
Copy link
Member

Thanks! I'll have to read through the code and check, but sounds reasonable.

@cowtowncoder
Copy link
Member

Looks good. Before I merge this, there is one formality we need; the Contributor License Agreement.
Before first contribution, we need you to print and fill this:

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

and then scan, email to info at fasterxml dot com.

This is unfortunately necessary so we can distribute the code and so forth. But it only needs to be sent once and then it's good for any number of contributions.

Thank you once again for the contribution, looking forward to merging it!

@flozano
Copy link
Contributor Author

flozano commented Feb 13, 2015

sent!

cowtowncoder added a commit that referenced this pull request Feb 13, 2015
@cowtowncoder cowtowncoder merged commit 7cf964a into FasterXML:master Feb 13, 2015
@flozano flozano deleted the fix/703 branch February 13, 2015 19:07
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