search

preactjs / signals

Manage state with style in every framework
https://preactjs.com/blog/introducing-signals/
MIT License
3.62k stars 88 forks source link

chore(hooks): removed usage of useMemo instead using useConst implemented with useRef #543

Open danilocastro-shopify opened 3 months ago

danilocastro-shopify commented 3 months ago

Proposing to prevent using hooks based on array dependency to simplify data lifecycle.

Maybe this change is small and not worthy. Considering the usage of several hooks around the application, it would reduce the GC time slices.

changeset-bot[bot] commented 3 months ago

⚠️ No Changeset found

Latest commit: 1f4aba507af3e75c982c473d2e87ead9aac628cc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

netlify[bot] commented 3 months ago

Deploy Preview for preact-signals-demo failed.

Name Link
Latest commit 1f4aba507af3e75c982c473d2e87ead9aac628cc
Latest deploy log https://app.netlify.com/sites/preact-signals-demo/deploys/6605984058bb800008446f4a