Repository navigation
Need to update cachetools decorator type stub with paramspec #4849
Description
In https://github.com/python/typeshed/blob/master/third_party/2and3/cachetools/decorators.pyi, ParamSpec needs to be added to the Callable and ContextManager params
def cached(
cache: Optional[MutableMapping[_KT, _VT]], key: Callable[..., _KT] = ..., lock: Optional[ContextManager[_T_co]] = ...
) -> Callable[[_T], _T]: ...
Activity
srittau commentedon Dec 28, 2020
Depends on #4827.
AlexWaygood commentedon Jun 12, 2022
ParamSpeccan now be used in most situations, but I'm not immediately sure how thecachetoolsstub can be improved withParamSpec. (PRs to improve ourcachetoolsstubs are of course welcome!)The linked pylance issue seems to be to do with
functools.lru_cache, not our third-partycachetoolsstubs.AlexWaygood commentedon Jun 14, 2022
Closing as per my comments above, but feel free to open a new issue (or, better yet, a PR) if there are indeed still issues with our
cachetoolsstubs :)Issues around
functools.lru_cacheare being discussed in depth at #6347 and #7771 already.