Conversation
|
any update on this PR? #5943 (comment) |
|
Sorry, I didn't notice I was testing against some older version instead of the proposed upper bound 0.17.32. I pushed one more change needed for that version. A similar change was also proposed in #7967. That pull request was closed with "This is fixed in a later version of the library", but that is not exactly the case as I described in a newer upstream ticket https://sourceforge.net/p/ruamel-yaml/tickets/482/. |
The issue is actually fixed with ruamel-yaml 0.17.36, so I dropped the workaround and raised the lower bound for this dependency. |
* ruamel-yaml 0.17.22 comes with a style change that affects test results. From https://sourceforge.net/p/ruamel-yaml/code/ci/0.17.22/tree/CHANGES > plain scalars: put single words longer than width on a line of their own, instead of after the previous line * ruamel-yaml 0.17.29 introduces an issue with add_constructor and DEFAULT_MAPPING_TAG, and the issue is fixed with 0.17.36 [1], hence the new lower bound. [1] https://sourceforge.net/p/ruamel-yaml/tickets/482/
|
This pull request was part of efforts on keeping compatibility between aws-cli v2 and newer Python dependencies on Arch Linux. Now that I gave up maintaining aws-cli-v2 in Arch Linux repos (#8342 (comment)), the efforts are abandoned. |
2 participants
Issue #, if available:
N/A
Description of changes:
Some aws-cli changes are needed for ruamel-yaml 0.17.22 and 0.17.29. See individual commit messages.
This PR supersedes #8257
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.