Skip to content
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

Move back to mainline vcrpy with urllib3 2.x when possible #2456

Open
dgw opened this issue May 21, 2023 · 1 comment
Open

Move back to mainline vcrpy with urllib3 2.x when possible #2456

dgw opened this issue May 21, 2023 · 1 comment
Labels

Comments

@dgw
Copy link
Member

dgw commented May 21, 2023

Follow-up to #2455. We only want this pinned temporarily, as described in 6e64c96; when possible, the pin should be removed.

Relies on resolution of upstream issue, kevin1024/vcrpy#688

@dgw dgw added the Build label May 21, 2023
@dgw
Copy link
Member Author

dgw commented Dec 19, 2023

#2519 made unpinning urllib3 possible, and the upstream issue in vcrpy has been "closed". But we're still doing Kludgy Things related to both packages:

# use fork of vcrpy 5.x until kevin1024/vcrpy#777 is (hopefully) accepted
# (or until py3.9 EOL... in 10/2025, I HOPE NOT)
vcrpy @ git+https://github.com/sopel-irc/vcrpy@uncap-urllib3

I will retitle this issue to reflect the current state, which is that vcrpy might or might not ever officially support using urllib3 2.x on Python <3.10 given problems with vcrpy's test suite on those combinations. There's no guarantee that I'll ever manage to get my patch into a state acceptable to upstream (but the test failures happen only in the suite's "offline" mode that mocks out actual communication with a server, seeming to work just fine when talking to real servers).

@dgw dgw changed the title Unpin transitive dependency urllib3 Move back to mainline vcrpy with urllib3 2.x when possible Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant