Skip to content

Replace a number of default argument values with "..." #1280

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 1 commit into from
May 21, 2017

Conversation

JelleZijlstra
Copy link
Member

pytype apparently doesn't like default values that aren't ints or
None/True/False.

pytype apparently doesn't like default values that aren't ints or
None/True/False.
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG! (Note that it's subtle, you can't replace "= None" with "= ..." unless the type is already Optional[]. But I checked everything and it's good.)

@JelleZijlstra
Copy link
Member Author

Yes, I made sure to only remove = None for Optional arguments. In the future I'll do a pass of replacing arg: T = None with arg: Optional[T] = ... , assuming we move forward with python/typing#275.

@gvanrossum gvanrossum merged commit 728b977 into python:master May 21, 2017
@JelleZijlstra JelleZijlstra deleted the ellipsisdefaults branch May 21, 2017 22:07
li-dan pushed a commit to li-dan/typeshed that referenced this pull request May 22, 2017
pytype apparently doesn't like default values that aren't ints or
None/True/False.
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