Skip to content
This repository was archived by the owner on Jul 9, 2018. It is now read-only.

Conversation

@aduth
Copy link
Member

@aduth aduth commented Apr 11, 2018

Related: WordPress/gutenberg#5933

This pull request seeks to remove babel-plugin-lodash from the default Babel preset, instead expecting that a consumer would reference Lodash directly by script reference. At worst, the removal of this plugin would bundle all of Lodash into a generated bundle, rather than cherry-picking specific modules. For this reason, it's expected this will be published as a minor release (backward-compatible), not major.

This is needed for Gutenberg to continue to use @wordpress/babel-preset-default, else at least it must be configurable. But with the current direction of core to adopt Lodash as a provided vendor script, it's not expected to need to be an internally-used option to opt for the plugin.

Copy link
Member

@ntwb ntwb left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@aduth aduth merged commit 28b26c8 into master Apr 12, 2018
@aduth aduth deleted the remove/babel-plugin-lodash branch April 12, 2018 18:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants