Skip to content

Update to Psych 3.3.2#6680

Merged
headius merged 3 commits intojruby:masterfrom
headius:update_psych_ruby2.6
May 26, 2021
Merged

Update to Psych 3.3.2#6680
headius merged 3 commits intojruby:masterfrom
headius:update_psych_ruby2.6

Conversation

@headius
Copy link
Member

@headius headius commented May 20, 2021

Another attempt to update Psych appropriate for a Ruby 2.6 release.

@headius headius added this to the JRuby 9.3.0.0 milestone May 20, 2021
@headius
Copy link
Member Author

headius commented May 25, 2021

The failures here all appear to be due to a behavior change in SnakeYAML that causes it to defer some emits while waiting for comment events: https://bitbucket.org/asomov/snakeyaml/pull-requests/7

The tests in question can be fixed by adding @stream.finish but this behavior change in SnakeYAML may have other implications. I will exclude or modify the tests in this PR and file issues with Psych and SnakeYAML about it.

@headius
Copy link
Member Author

headius commented May 25, 2021

The failures here all appear to be due to a behavior change in SnakeYAML

Correction: the empty output failures are due to this behavior change (ruby/psych#499, https://bitbucket.org/asomov/snakeyaml/issues/503/comment-handling-causes-deferred-emit), while the emitter exceptions are due to ruby/psych#500.

@headius headius merged commit d6e5a46 into jruby:master May 26, 2021
@headius headius deleted the update_psych_ruby2.6 branch May 26, 2021 02:08
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.

1 participant