fix(poetry): Fix existing range unions

[nphilipp]

Previously, the call to .replace() only ever affected the first occurrence of "||" in the string. This makes the regex global to remove all spurious commas.

Fixes: #18130

Signed-off-by: Nils Philippsen nils@redhat.com

Changes

Widening Python version range unions adds syntactically incorrect commas, this change fixes it and adds a test.

Context

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

• Code inspection only, or
• Newly added/modified unit tests, or
• No unit tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[rarkins]

@nphilipp Thanks for your patience and PR. If you can find out if there's anything particular about this CLA that's problematic, please let me know

[renovate-release]

🎉 This PR is included in version 34.99.1 🎉

The release is available on:

• GitHub release
• 34.99.1

Your semantic-release bot 📦🚀

[nphilipp]

@nphilipp Thanks for your patience and PR. If you can find out if there's anything particular about this CLA that's problematic, please let me know

No worries, the blame for the delay lies entirely on my end 😉. I'll get in touch with our legal team and, if it's alright with you, will try to establish a contact between you and them.