Add option ForbidDuplicateKeys

ingydotnet / yaml-libyaml-pm

Perl Binding to libyaml

http://search.cpan.org/dist/YAML-LibYAML/

33 stars 37 forks source link

Closed perlpunk closed 2 years ago

perlpunk commented 2 years ago

By default it is false as it could suddenly break code.

See issue #17

YAML::PP forbids duplicate keys by default, according to the YAML Spec.

perlpunk commented 2 years ago

Additional suggestion: When setting the option to undef, it will issue a warning, and we can make undef the default at some point in the future.

perlpunk commented 2 years ago

The failing CI should be fixed as soon as https://github.com/ingydotnet/zilla-dist-pm/pull/59 is released

ingydotnet commented 2 years ago

Additional suggestion: When setting the option to undef, it will issue a warning, and we can make undef the default at some point in the future.

Let's not do that now. See review comment about config object.