Skip to content
This repository was archived by the owner on Aug 14, 2024. It is now read-only.

Memoize payload API#53

Merged
fraxken merged 1 commit into
mainfrom
In_memory_memoize
Mar 16, 2023
Merged

Memoize payload API#53
fraxken merged 1 commit into
mainfrom
In_memory_memoize

Conversation

@fabnguess
Copy link
Copy Markdown
Contributor

This PR aims to resolve issue #14

  • Added memoize API
  • Added memoized API

@fabnguess fabnguess requested a review from fraxken February 17, 2023 20:17
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread test/memoize.spec.ts Outdated
Comment thread package.json Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
@fraxken
Copy link
Copy Markdown
Member

fraxken commented Feb 25, 2023

You also need to add documentation.

UT also need to be improved.

Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread test/memoized.spec.ts Outdated
Comment thread test/memoized.spec.ts Outdated
Comment thread README.md Outdated
@fabnguess fabnguess force-pushed the In_memory_memoize branch from 66cf31a to eecf96d Compare March 1, 2023 10:28
Comment thread src/functions/memoize.ts Outdated
Comment thread README.md Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts Outdated
Comment thread src/functions/memoize.ts
@fraxken fraxken merged commit 085b3c9 into main Mar 16, 2023
@fraxken
Copy link
Copy Markdown
Member

fraxken commented Mar 16, 2023

@all-contributors please add @fabnguess for code, maintenance

@allcontributors
Copy link
Copy Markdown
Contributor

@fraxken

I've put up a pull request to add @fabnguess! 🎉

@fraxken fraxken deleted the In_memory_memoize branch March 16, 2023 09:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants