Skip to content
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

Don't use the same memoized function across instances when .decorator() is used #75

Merged
merged 5 commits into from
Apr 20, 2021

Conversation

Richienb
Copy link
Contributor

Fixes: #74

Signed-off-by: Richie Bendall <richiebendall@gmail.com>
readme.md Outdated Show resolved Hide resolved
@fregante fregante added the bug label Apr 20, 2021
sindresorhus and others added 2 commits April 20, 2021 17:03
Signed-off-by: Richie Bendall <richiebendall@gmail.com>
@Richienb Richienb changed the title Don't memoize functions by default across instances when .decorator() is used Don't memoize functions across instances when .decorator() is used Apr 20, 2021
@Richienb Richienb changed the title Don't memoize functions across instances when .decorator() is used Don't memoize functions across instances when memoized with .decorator() Apr 20, 2021
@Richienb Richienb changed the title Don't memoize functions across instances when memoized with .decorator() Don't memoize functions across instances when .decorator() is used Apr 20, 2021
@Richienb Richienb changed the title Don't memoize functions across instances when .decorator() is used Don't use the same memoized function across instances when .decorator() is used Apr 20, 2021
index.ts Outdated Show resolved Hide resolved
Richienb and others added 2 commits April 20, 2021 23:42
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
@Richienb Richienb requested a review from sindresorhus April 20, 2021 11:44
@sindresorhus sindresorhus merged commit d21357c into sindresorhus:main Apr 20, 2021
@sindresorhus
Copy link
Owner

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Values are memoised across class/prototype instances
3 participants