Update Unused parameters in stubs/#9704
Uh oh!
There was an error while loading. Please reload this page.
Conversation
This comment has been minimized.
This comment has been minimized.
According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉 |
| # def __new__.<locals>.getPlug.<locals>.plug | ||
| @type_check_only | ||
| defplug(self, *args: object, **kw: object): ... | ||
| defplug(self, *args: Unused, **kw: Unused) ->NoReturn: ... |
There was a problem hiding this comment.
I'm not sure I understand why this method is in the stub at all.
There was a problem hiding this comment.
Pretty sure it's just to use as an alias below. Source: https://github.com/pyasn1/pyasn1/blob/main/pyasn1/type/base.py#L212 (which the comment also point to being defined in the function getPlug in the function __new__)
| def__delitem__(self, *arg: Unused, **kw: Unused) ->NoReturn: ... | ||
| def__setitem__(self, *arg: Unused, **kw: Unused) ->NoReturn: ... | ||
| def__setattr__(self, *arg: Unused, **kw: Unused) ->NoReturn: ... |
There was a problem hiding this comment.
(I haven't looked at the source code here, just speculating.)
If instances of ImmutableContainer are meant to be, well, immutable, it might produce better type checking if we just omitted these methods from the stub altogether.
There was a problem hiding this comment.
Maybe relevant to your question:
microsoft/pyright#4653 (comment)
python/mypy#14726
3 participants
Follow-up to #9519 for third-party stubs now that a new version of mypy has been released
Ref #9297
Avoids touching
__a?exit__. That's in #9696Three steps:
Unusedinstead of_Unusedobject | Nonetoobject(unless there's a helpful reason for it, in which case a comment was added)objectto be typed withUnused