-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
FIx various typos #11498
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
FIx various typos #11498
Conversation
Found via `codespell -q 3 -S ./mypyc/external/googletest -L alo,ans,asend,ba,bre,dne,fo,haa,ist,larg,myu,nams,spawnve,statics,thow,ure,wont`
Please review f7798ec closely |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT, thanks! 👍
@@ -121,7 +121,7 @@ class type(object): | |||
def __new__(cls, name: str, bases: Tuple[type, ...], namespace: Dict[str, Any]) -> type: ... | |||
def __call__(self, *args: Any, **kwds: Any) -> Any: ... | |||
def __subclasses__(self: _TT) -> List[_TT]: ... | |||
# Note: the documentation doesnt specify what the return type is, the standard | |||
# Note: the documentation doesn't specify what the return type is, the standard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you make these changes in the separate typeshed repo instead? Mypy just copies it in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, what's the repo address?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/python/typeshed, thanks! Please remove the typeshed changes from this PR, we'll carry them in the next time we sync typeshed into this repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted in 40e1fd8
Found via `codespell -q 3 -L alo,asend,ba,nams,spawnve ` Originally found downstream python/mypy#11498 (comment)
Found via `codespell -q 3 -L alo,asend,ba,nams,spawnve` Originally found downstream python/mypy#11498 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks great!
# Include import froms that import names defined in __all__. | ||
# Include import forms that import names defined in __all__. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the original version was correct here. This is referring to "import froms" as in from x import y
statements, not import forms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
As pointed out by @pranavrajpal in #11498 (comment)
As pointed out by @pranavrajpal in #11498 (comment)
Found via `codespell -q 3 -S ./mypyc/external/googletest -L alo,ans,asend,ba,bre,dne,fo,haa,ist,larg,myu,nams,spawnve,statics,thow,ure,wont`
As pointed out by @pranavrajpal in python#11498 (comment)
Found via
codespell -q 3 -S ./mypyc/external/googletest -L alo,ans,asend,ba,bre,dne,fo,haa,ist,larg,myu,nams,spawnve,statics,thow,ure,wont