preactjs / signals

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

Use function expression name if it has one #446

Closed andrewiggins closed 10 months ago

andrewiggins commented 10 months ago

Component's like

var render = function App() { ... }

will now by transformed by the babel plugin since the function expression has a valid Component name.

Before it would not be transformed because the variable name was not valid because the variable name begins with a lowercase letter.

changeset-bot[bot] commented 10 months ago

đŸĻ‹ Changeset detected

Latest commit: ca716b35bfab1981e6d41033c5b19309d8164524

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-transform | 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 10 months ago

Deploy Preview for preact-signals-demo ready!

Name Link
Latest commit ca716b35bfab1981e6d41033c5b19309d8164524
Latest deploy log https://app.netlify.com/sites/preact-signals-demo/deploys/655f170ad5c27700083fd7a9
Deploy Preview https://deploy-preview-446--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.

github-actions[bot] commented 10 months ago

Size Change: +12 B (0%)

Total Size: 83.9 kB

Filename Size Change
packages/react-transform/dist/signals-*********.js 4.77 kB +4 B (0%)
packages/react-transform/dist/signals-transform.mjs 4.02 kB +4 B (0%)
packages/react-transform/dist/signals-transform.umd.js 4.88 kB +4 B (0%)
ℹī¸ View Unchanged | Filename | Size | | :--- | :---: | | `docs/dist/assets/client.********.js` | 46.8 kB | | `docs/dist/assets/index.********.js` | 1.07 kB | | `docs/dist/assets/jsxRuntime.module.********.js` | 281 B | | `docs/dist/assets/preact.module.********.js` | 4.02 kB | | `docs/dist/assets/signals-core.module.********.js` | 1.46 kB | | `docs/dist/assets/signals.module.********.js` | 2.02 kB | | `docs/dist/assets/style.********.js` | 21 B | | `docs/dist/assets/style.********.css` | 1.21 kB | | `docs/dist/basic-********.js` | 244 B | | `docs/dist/demos-********.js` | 3.41 kB | | `docs/dist/nesting-********.js` | 1.13 kB | | `docs/dist/react-********.js` | 239 B | | `packages/core/dist/signals-core.js` | 1.54 kB | | `packages/core/dist/signals-core.mjs` | 1.56 kB | | `packages/preact/dist/signals.js` | 1.27 kB | | `packages/preact/dist/signals.mjs` | 1.22 kB | | `packages/react/dist/signals.js` | 1.43 kB | | `packages/react/dist/signals.mjs` | 1.34 kB |

compressed-size-action