Apply @mcepl's tests_overcome_bpo42967.patch

gruns / furl

🌐 URL parsing and manipulation made easy.

Other

2.6k stars 151 forks source link

Apply @mcepl's tests_overcome_bpo42967.patch #140

Closed bnavigator closed 3 years ago

bnavigator commented 3 years ago

From https://github.com/gruns/furl/issues/135#issuecomment-784331450

Together with #139 this should close #135.

Didn't address https://github.com/gruns/furl/issues/135#issuecomment-793076723. The assertions still work there

dvzrv commented 3 years ago

@gruns why was this not included in 2.1.1? This patch and #139 now have to be applied on multiple downstreams. Can you include them and #141 in another patch level release? That would be very helpful!

gruns commented 3 years ago

merged!

huge thank you for your help and PRs, ben! :raised_hands:

now live in the new furl v2.1.2