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

Use something other than YAML #170

Open
robrwo opened this issue Jan 2, 2024 · 2 comments
Open

Use something other than YAML #170

robrwo opened this issue Jan 2, 2024 · 2 comments

Comments

@robrwo
Copy link

robrwo commented Jan 2, 2024

YAML 1.31 recommends using something else, such as YAML::PP.

I have experimented with switching to YAML::PP and YAML::Tiny but the tests fail to subtle differences in serialisation. Notably, some numbers are turned into strings, e.g. version: \'1.64\' instead of version: 1.64.

In experiments YAML::Tiny seems to work better as it preserves decimal places, e.g. 0.00 is not turned into 0.

(On closer inspection, YAML::Tiny does not support all of the required features.)

@robrwo
Copy link
Author

robrwo commented Jan 2, 2024

I'm unsure if using an XS module or one that uses an external library like YAML::Syck is appropriate.

@rabbiveesh
Copy link
Contributor

I don't think i'd want to make an XS module a mandatory dep, though i'd be happy for it to be an optional one that's loaded if available.

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

No branches or pull requests

2 participants