react-spring icon indicating copy to clipboard operation
react-spring copied to clipboard

fix: add annotations for tree shaking

Open CodyJasonBennett opened this issue 2 years ago • 2 comments

Why

Implements #1158.

What

Adds pure annotations to tree-shake top-level expressions. This can be done at build-time with an AST such as from Babel. Need to verify whether built-ins like Symbol#for and Set/WeakMap constructors are picked up as pure. Same with mutations in Rafz.

Checklist

  • [ ] Documentation updated
  • [ ] Demo added
  • [ ] Ready to be merged

CodyJasonBennett avatar Sep 16 '23 00:09 CodyJasonBennett

⚠️ No Changeset found

Latest commit: 2f43f09a8b0305c97790d1d41a6aca27479e5a66

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

changeset-bot[bot] avatar Sep 16 '23 00:09 changeset-bot[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-spring ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 16, 2023 0:04am

vercel[bot] avatar Sep 16 '23 00:09 vercel[bot]