Rename UniMemoize#atLeast
#742
michalszynkiewicz
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
|
Do you have other suggestions for the whole memoization API? |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
Nothing more comes to my mind. |
Beta Was this translation helpful? Give feedback.
-
|
Moved to #743 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
For me,
atLeastsuggests that the value will be memoized/cached for the specified amount of time or longer (I'd guess might be stored in a weak reference after the period of time).The discussion on a Stork PR says that the method is actually specifying time to live for the cached value.
Could the method be renamed to e.g.
forDuration?CC @jponge
Beta Was this translation helpful? Give feedback.
All reactions