cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

Allows tree shaking unused modules

Open zomars opened this issue 3 years ago • 1 comments

What does this PR do?

  • Marks all packages as "side effect" free which all SHOULD be unless I'm mistaken
  • This will allow correct behavior for webpack tree shaking
  • Noticed some minor gains in some pages locally (up to 20kb of bundle size reduction)

Type of change

  • [x] Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • [ ] Before checking out this branch: yarn build
  • [ ] After checking out this branch yarn build
  • [ ] Compare overall bundle size
  • [ ] yarn test-e2e

zomars avatar Aug 09 '22 12:08 zomars

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

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Aug 11, 2022 at 1:06AM (UTC)
cal-com ❌ Failed (Inspect) Aug 11, 2022 at 1:06AM (UTC)
nightly-cal ❌ Failed (Inspect) Aug 11, 2022 at 1:06AM (UTC)
1 Ignored Deployment
Name Status Preview Updated
swagger ⬜️ Ignored (Inspect) Aug 11, 2022 at 1:06AM (UTC)

vercel[bot] avatar Aug 09 '22 12:08 vercel[bot]