-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
dvc.yaml: ruamel.yaml 0.17.22 oddly splits long strings #9397
Comments
probably related to the change for https://sourceforge.net/p/ruamel-yaml/tickets/427/ ruamel has a hidden default width of 80, so it looks like it will force wrapping if the line has a string token > length 80 now. At least in our tests we probably need to explicitly set |
@pmrowla Thanks for the research! 🙏 Very good suggestion! Surprised by this lossy folding and the linked issue suggests that this might not be solved any time soon. I'll just adjust the test for now. |
Just modified the test for now as it was pretty fragile anyway. |
ruamel.yaml 0.17.22:
vs old:
Which is causing our CI to fail on tests/func/test_lockfile.py::test_cmd_changes_other_orders_are_preserved
The text was updated successfully, but these errors were encountered: