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

Relax requests version spec #274

Closed
wants to merge 13 commits into from
Closed

Relax requests version spec #274

wants to merge 13 commits into from

Conversation

janhurst
Copy link

@janhurst janhurst commented Mar 6, 2023

Pinning requests to <2.22 causes many packages to conflict.

See #272

@shicholas
Copy link

Hello, anything I can do to help this PR? Thanks!

@janhurst
Copy link
Author

janhurst commented Aug 7, 2023

I think this has effectively been implemented already

@shicholas
Copy link

I tried installing version 12.1.0 and still couldn't upgrade our requests dependency. As a result dependabot is flagging our repositories that use this library.

@janhurst
Copy link
Author

janhurst commented Aug 7, 2023

This tag in the repo seems to be setting the requests dependency to requests<3 which is at least an improvement from the old pin to to requests<2.22

@janhurst janhurst closed this Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants