search

ruby / psych

A libyaml wrapper for Ruby
MIT License
566 stars 205 forks source link

Remove SnakeYAML settings that don't work #653

Closed headius closed 1 year ago

headius commented 1 year ago

These settings are not enforced at the parser level in SnakeYAML Engine, instead being enforced during the construction of the YAML node graph. Because the JRuby Psych wrapper around SnakeYAML only uses the parser directly, the settings have no effect. This commit removes the ineffective settings until we can decide what to do about them.

See #613, #647, and #649.

headius commented 1 year ago

@hsbt This is merged and could be released. Let me know if you will do it or if I should do it.

headius commented 1 year ago

@tenderlove @hsbt Can we release this week? JRuby 9.4.4.0 should be next week and I'd like to get this update in there.

hsbt commented 1 year ago

@headius I released Psych 5.1.1 with this fix. Let me know if you have another request.