Skip to content

Add feature to support quoted field names in YAML writer. #69

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

Conversation

baconmania
Copy link

Fixes #68.

/cc @cowtowncoder

@cowtowncoder
Copy link
Member

Looks good, and although adding features in patches is usually no-no, I think I'll make exception here because 2.9 will probably be the last 2.x version (and conversely 3.0 will take quite a while to be released).

Just one process thing: unless I've asked for and received it, we need CLA before the first merged contribution. It's one time, one page document from:

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

and usual way is to print, fill & sign, scan, email to info at fasterxml dot com.
Once it is received I can merge this PR as well as any other contributions.
Apologies for the hassle.

Thank you again for reporting this & contribution the fix!

@cowtowncoder
Copy link
Member

@baconmania hiya! I would really like to merge this, but would need CLA if you get a chance.

@cowtowncoder cowtowncoder added yaml Issue related to YAML format backend active labels Apr 11, 2018
@cowtowncoder
Copy link
Member

Implemented in slightly different, as per #68.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
yaml Issue related to YAML format backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants