go-memoize icon indicating copy to clipboard operation
go-memoize copied to clipboard

feat: Adds a way to use generics with this library.

Open jfyne opened this issue 2 years ago • 2 comments

I am very open to changing this to another way, but I think this is pretty clean. Thanks for the library.

jfyne avatar Jan 12 '23 09:01 jfyne

This will also not break any existing code

jfyne avatar Jan 12 '23 09:01 jfyne

@kofalt can this be merged!?

delijati avatar Jul 18 '23 11:07 delijati

@kofalt , it would be great to include this generic feature as an evolution! The memoize.Call is also a better Go style name for the function, since the package name already has "memoize"

natenho avatar May 04 '24 19:05 natenho

Hi there @natenho, thanks for the ping. I will have a look at this tomorrow.

kofalt avatar May 04 '24 21:05 kofalt

Thanks for the PR @jfyne, I must not have had email notifications on when this was originally sent in.

I am merging with a second small change to bump the go version.

kofalt avatar May 06 '24 04:05 kofalt