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

SnakeYAML Engine #106

Merged
merged 8 commits into from
Feb 28, 2019
Merged

SnakeYAML Engine #106

merged 8 commits into from
Feb 28, 2019

Conversation

asomov
Copy link
Contributor

@asomov asomov commented Sep 8, 2018

Hi all,
finally the SnakeYAML Engine has been released.
It is a re-build of SnakeYAML which supports YAML 1.2 (superset of JSON)

This pull request introduces SnakeYAML Engine and makes jackson 'YAML 1.2 complaint'.

@cowtowncoder
Copy link
Member

@asomov Apologies for missing this one. I'll see if I can manually merge this in master for 3.0 (looks like Java 8 is required)

@cowtowncoder cowtowncoder added active 3.x yaml Issue related to YAML format backend labels Feb 27, 2019
@asomov
Copy link
Contributor Author

asomov commented Feb 27, 2019

I will check the merge conflicts.

(Yes, java 8 is required)

# Conflicts:
#	yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLFactory.java
#	yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLGenerator.java
#	yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLParser.java
#	yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/snakeyaml/error/MarkedYAMLException.java
@asomov
Copy link
Contributor Author

asomov commented Feb 27, 2019

The merge conflicts were resolved.

@cowtowncoder
Copy link
Member

@asomov excellent, thank you!

@cowtowncoder cowtowncoder merged commit c258dac into FasterXML:master Feb 28, 2019
@cowtowncoder cowtowncoder modified the milestones: 3.0., 3.0.0 Feb 28, 2019
cowtowncoder added a commit that referenced this pull request Feb 28, 2019
@asomov asomov deleted the snakeyaml-engine branch February 28, 2019 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x yaml Issue related to YAML format backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants