Skip to content

Conversation

@albestro
Copy link
Contributor

@albestro albestro commented Nov 19, 2025

Fix #272

This PR starts as a reproducer of the #272 (with a trivial test) and it becomes a tentative in fixing it.

  • Fix ruamel.yaml dump function usage

@albestro
Copy link
Contributor Author

We can bikeshed on the load change, but the real point is how to refactor in order to comply with the different dump API, which does not return strings (there is a simple workaround with StringIO, but I think it might be better to change a bit the paradigm).

Before investing time in a proper change, I quickly addressed the problem to give it a try and check if everything worked. I've not checked that scientifically (yet), but I was able to build a uenv image with this version.

If and when we decide to proceed on this path, I'm going to properly take care of the changes needed.

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.

Duplicate entries in YAML mappings (dicts) gets implicitly overwritten

1 participant