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 werkzeug to v1.0.1 #178

Merged
merged 1 commit into from
Jun 1, 2021
Merged

Conversation

adamchel
Copy link
Contributor

@adamchel adamchel commented May 12, 2021

In Werkzeug's latest 2.0.0 release, support for Python 2 was removed: pallets/werkzeug#1693.

This means that the imports from werkzeug._compat in serverless_wsgi.py no longer work:

from werkzeug._compat import BytesIO, string_types, to_bytes, wsgi_encoding_dance

This can be fixed by pinning the werkzeug dependency to a pre-v2.0.0 version until support for the werkzeug._compat module is no longer necessary.

@codecov
Copy link

codecov bot commented May 12, 2021

Codecov Report

Merging #178 (cc3b214) into master (a1a1c55) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   99.13%   99.13%           
=======================================
  Files           5        5           
  Lines         575      575           
  Branches       69       69           
=======================================
  Hits          570      570           
  Misses          5        5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1a1c55...cc3b214. Read the comment docs.

@adamchel
Copy link
Contributor Author

The force-push was to change the author of the commit to my personal email address, instead of my work email address.

@logandk
Copy link
Owner

logandk commented Jun 1, 2021

Thanks, @adamchel!

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.

2 participants