Skip to content

Conversation

@FlorianSW
Copy link
Contributor

Instead of importing top-level module, consistently use deep-imports for lodash. This allows projects to properly tree-shake lodash in their build is substantially reduce the size of the resulting bundle.

Instead of importing top-level module, consistently use
deep-imports for lodash. This allows projects to properly
tree-shake lodash in their build is substantially reduce
the size of the resulting bundle.
@CLAassistant
Copy link

CLAassistant commented Dec 12, 2024

CLA assistant check
All committers have signed the CLA.

@netlify
Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit 49cccf8
🔍 Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/675b479e780c300008b49703
😎 Deploy Preview https://deploy-preview-2406--jsonforms-examples.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.

@coveralls
Copy link

Coverage Status

coverage: 82.504%. remained the same
when pulling 49cccf8 on FlorianSW:lodash/tree-shake
into 21400a8 on eclipsesource:master.

@lucas-koehler lucas-koehler self-requested a review December 13, 2024 08:47
Copy link
Contributor

@lucas-koehler lucas-koehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @FlorianSW , thanks for this improvement ❤️
The changes LGTM 👍

@lucas-koehler lucas-koehler merged commit 9c1c7da into eclipsesource:master Dec 13, 2024
8 checks passed
@lucas-koehler lucas-koehler added this to the 3.5 milestone Dec 13, 2024
@FlorianSW FlorianSW deleted the lodash/tree-shake branch December 14, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants