Need to update cachetools decorator type stub with paramspec

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]: ...

Depends on #4827.

ParamSpec can now be used in most situations, but I'm not immediately sure how the cachetools stub can be improved with ParamSpec. (PRs to improve our cachetools stubs are of course welcome!)

The linked pylance issue seems to be to do with functools.lru_cache, not our third-party cachetools stubs.

ParamSpec can now be used in most situations, but I'm not immediately sure how the cachetools stub can be improved with ParamSpec. (PRs to improve our cachetools stubs are of course welcome!)

The linked pylance issue seems to be to do with functools.lru_cache, not our third-party cachetools stubs.

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 cachetools stubs :)

Issues around functools.lru_cache are being discussed in depth at #6347 and #7771 already.

Navigation Menu

Repository navigation

Need to update cachetools decorator type stub with paramspec #4849

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Need to update cachetools decorator type stub with paramspec #4849

Description

Activity

srittau commented on Dec 28, 2020

AlexWaygood commented on Jun 12, 2022

AlexWaygood commented on Jun 14, 2022

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions