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

Pin requests to <2.30.0 to fix test failures #95

Merged
merged 1 commit into from
May 3, 2023

Conversation

AlexWaygood
Copy link
Contributor

requests v2.30.0 has just been released: https://github.com/psf/requests/releases/tag/v2.30.0

It appears to be causing the stub-uploader tests to fail on the typeshed main branch:

I haven't looked into what's causing the failures at all, but here's a stopgap PR to pin requests and stop the tests from failing

`requests v2.30.0` has just been released: https://github.com/psf/requests/releases/tag/v2.30.0

It appears to be causing the stub-uploader tests to fail on the typeshed `main` branch:
  - https://github.com/python/typeshed/actions/runs/4873926053/jobs/8694191409
  - https://github.com/python/typeshed/actions/runs/4873908728/jobs/8694149563

I haven't looked into what's causing the failures at all, but here's a stopgap PR to pin `requests` and stop the tests from failing
@AlexWaygood AlexWaygood changed the title Pin requests to <2.30.0 Pin requests to <2.30.0 to fix test failures May 3, 2023
@JelleZijlstra JelleZijlstra merged commit 2e7d070 into typeshed-internal:main May 3, 2023
@AlexWaygood AlexWaygood deleted the patch-1 branch May 3, 2023 16:53
@AlexWaygood
Copy link
Contributor Author

Confirmed that typeshed's main branch is green again: https://github.com/python/typeshed/actions/runs/4873926053/jobs/8695489553

@hauntsaninja
Copy link
Contributor

Wow. Looks like they had to yank requests 2.30 psf/requests#6439

@AlexWaygood
Copy link
Contributor Author

Ouch

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.

3 participants