-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[HOLD for payment 2024-08-05] [TS] Memoize utility instance lacks typings for cache property #46069
Comments
I will work on this! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.13-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-08-05. 🎊 |
Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
I think we can close this one, right @mountiny? |
Yep, thanks! |
Details
This is a known regression, which originates from #43868
Instance of
memoize
module lacks correct typings forcache
property. Methods of cache (get
,set
, etc.) havekey
parameter typed asunknown
.Steps to reproduce
memoize
by callingmemoize
methodcache
property methods on the instanceExpected behaviour
key
argument in cache methods matches arguments of memoized functionActual behaviour
key
argument is set tounknown
The text was updated successfully, but these errors were encountered: