Skip to content

Improve werkzeug stubs #1730

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

Merged
merged 10 commits into from
Mar 29, 2018
Merged

Improve werkzeug stubs #1730

merged 10 commits into from
Mar 29, 2018

Conversation

JelleZijlstra
Copy link
Member

Removes a few default values that were not ...; adds missing types; harmonizes the stub between Python 2 and 3. We should merge the 2 and 3 stubs for this library, but that's a bigger project.

Mostly worked off https://github.com/pallets/werkzeug/blob/master/werkzeug/wrappers.py.

@ambv
Copy link
Contributor

ambv commented Feb 14, 2018

Rebase and I'll take a look.

@ambv
Copy link
Contributor

ambv commented Mar 18, 2018

MutableMapping definition is missing.

@JelleZijlstra
Copy link
Member Author

Sorry for the bad merge, it's fixed now.

@JelleZijlstra
Copy link
Member Author

@ambv could you take another look?

@ambv
Copy link
Contributor

ambv commented Mar 29, 2018

LGTM.

@ambv ambv merged commit 0c15e5b into python:master Mar 29, 2018
@JelleZijlstra JelleZijlstra deleted the werkzeug branch March 29, 2018 02:40
yedpodtrzitko pushed a commit to yedpodtrzitko/typeshed that referenced this pull request Jan 23, 2019
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