preactjs / signals

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

Re-export useSignals in @preact/signals-react #474

Closed JonAbrams closed 8 months ago

JonAbrams commented 9 months ago

Fixes #473

changeset-bot[bot] commented 9 months ago

🦋 Changeset detected

Latest commit: ea8e3f06d1a36dc69e45cc0d7c56b48aa4f7f2e4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | --------------------- | ----- | | @preact/signals-react | Minor |

Not sure what this means? Click here to learn what changesets are.

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

netlify[bot] commented 9 months ago

Deploy Preview for preact-signals-demo ready!

Name Link
Latest commit ea8e3f06d1a36dc69e45cc0d7c56b48aa4f7f2e4
Latest deploy log https://app.netlify.com/sites/preact-signals-demo/deploys/65853221cf62e60008c696c0
Deploy Preview https://deploy-preview-474--preact-signals-demo.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

andrewiggins commented 8 months ago

Actually, just saw that the Readme was updated. Let's roll with that for now. Thanks for opening this though.

rschristian commented 8 months ago

@andrewiggins react-transform doesn't re-export core though, do we want this?

import { signal, effect } from '@preact/signals-react';
import { useSignals, useSignal } from '@preact/signals-react-runtime';

I'm not super in love with this myself.