Skip to content

Commit 7db5978

Browse files
cleanup + fix subtypes
1 parent 93bc08c commit 7db5978

File tree

5 files changed

+21
-7
lines changed

5 files changed

+21
-7
lines changed

stdlib/collections/__init__.pyi

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,8 @@ class OrderedDict(dict[_KT, _VT]):
384384
@overload
385385
def pop(self, key: object) -> _VT: ...
386386
@overload
387+
def pop(self, key: object, default: _VT) -> _VT: ...
388+
@overload
387389
def pop(self, key: object, default: _T) -> _VT | _T: ...
388390
def __eq__(self, value: object, /) -> bool: ...
389391
@overload
@@ -469,9 +471,11 @@ class ChainMap(MutableMapping[_KT, _VT]):
469471
@overload
470472
def setdefault(self, key: _KT, default: _VT) -> _VT: ...
471473
@overload
472-
def pop(self, key: _KT) -> _VT: ...
474+
def pop(self, key: Any) -> _VT: ...
475+
@overload
476+
def pop(self, key: Any, default: _VT) -> _VT: ...
473477
@overload
474-
def pop(self, key: _KT, default: _T) -> _VT | _T: ...
478+
def pop(self, key: Any, default: _T) -> _VT | _T: ...
475479
def copy(self) -> Self: ...
476480
__copy__ = copy
477481
# All arguments to `fromkeys` are passed to `dict.fromkeys` at runtime,

stdlib/mailbox.pyi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ class Mailbox(Generic[_MessageT]):
8888
def __len__(self) -> int: ...
8989
def clear(self) -> None: ...
9090
@overload
91-
def pop(self, key: str, default: None = None) -> _MessageT | None: ...
91+
def pop(self, key: str, default) -> _MessageT | None: ...
9292
@overload
9393
def pop(self, key: str, default: _T) -> _MessageT | _T: ...
9494
def popitem(self) -> tuple[str, _MessageT]: ...

stdlib/multiprocessing/managers.pyi

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,8 @@ if sys.version_info >= (3, 13):
8888
@overload
8989
def pop(self, key: _KT, /) -> _VT: ...
9090
@overload
91+
def pop(self, key: _KT, default: _VT, /) -> _VT: ...
92+
@overload
9193
def pop(self, key: _KT, default: _T, /) -> _VT | _T: ...
9294
def keys(self) -> list[_KT]: ... # type: ignore[override]
9395
def items(self) -> list[tuple[_KT, _VT]]: ... # type: ignore[override]
@@ -114,6 +116,8 @@ else:
114116
@overload
115117
def pop(self, key: _KT, /) -> _VT: ...
116118
@overload
119+
def pop(self, key: _KT, default: _VT, /) -> _VT: ...
120+
@overload
117121
def pop(self, key: _KT, default: _T, /) -> _VT | _T: ...
118122
def keys(self) -> list[_KT]: ... # type: ignore[override]
119123
def items(self) -> list[tuple[_KT, _VT]]: ... # type: ignore[override]

stdlib/os/__init__.pyi

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -719,6 +719,8 @@ class _Environ(MutableMapping[AnyStr, AnyStr], Generic[AnyStr]):
719719
@overload
720720
def pop(self, key: AnyStr) -> AnyStr: ...
721721
@overload
722+
def pop(self, key: AnyStr, default: AnyStr) -> AnyStr: ...
723+
@overload
722724
def pop(self, key: AnyStr, default: _T) -> AnyStr | _T: ...
723725
def setdefault(self, key: AnyStr, value: AnyStr) -> AnyStr: ...
724726
def copy(self) -> dict[AnyStr, AnyStr]: ...

stdlib/weakref.pyi

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,11 @@ class WeakValueDictionary(MutableMapping[_KT, _VT]):
112112
def valuerefs(self) -> list[KeyedRef[_KT, _VT]]: ...
113113
def setdefault(self, key: _KT, default: _VT) -> _VT: ...
114114
@overload
115-
def pop(self, key: _KT) -> _VT: ...
115+
def pop(self, key: object) -> _VT: ...
116116
@overload
117-
def pop(self, key: _KT, default: _T) -> _VT | _T: ...
117+
def pop(self, key: object, default: _VT) -> _VT: ...
118+
@overload
119+
def pop(self, key: object, default: _T) -> _VT | _T: ...
118120
@overload
119121
def update(self, other: SupportsKeysAndGetItem[_KT, _VT], /, **kwargs: _VT) -> None: ...
120122
@overload
@@ -166,9 +168,11 @@ class WeakKeyDictionary(MutableMapping[_KT, _VT]):
166168
@overload
167169
def setdefault(self, key: _KT, default: _VT) -> _VT: ...
168170
@overload
169-
def pop(self, key: _KT) -> _VT: ...
171+
def pop(self, key: object) -> _VT: ...
172+
@overload
173+
def pop(self, key: object, default: _VT) -> _VT: ...
170174
@overload
171-
def pop(self, key: _KT, default: _T) -> _VT | _T: ...
175+
def pop(self, key: object, default: _T) -> _VT | _T: ...
172176
@overload
173177
def update(self, dict: SupportsKeysAndGetItem[_KT, _VT], /, **kwargs: _VT) -> None: ...
174178
@overload

0 commit comments

Comments
 (0)