-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lodash: Remove completely from @wordpress/list-reusable-blocks
package
#44567
Conversation
Size Change: +390 B (0%) Total Size: 1.27 MB
ℹ️ View Unchanged
|
@@ -1,7 +1,7 @@ | |||
/** | |||
* External dependencies | |||
*/ | |||
import { kebabCase } from 'lodash'; | |||
import { paramCase as kebabCase } from 'change-case'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you feel about changing the usages to paramCase
, too, eliminating the kebabCase
name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my experience, kebabCase
is a more common name than paramCase
, plus it's already known like that in Gutenberg and in the WP core (see _wp_to_kebab_case()
, so I kind of prefer to keep the alias.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I thought that "kebab case" is a silly name that only Lodash uses, but if Core and others use it too, it's probably more standard than I thought 🙂
What?
This PR removes all of the Lodash from the
@wordpress/list-reusable-blocks
package, including thelodash
dependency altogether. There's just a single usage and it is pretty straightforward.Why?
Lodash is known to unnecessarily inflate the bundle size of packages, and in most cases, it can be replaced with native language functionality. See these for more information and rationale:
@wordpress/api-fetch
package haslodash
as a dependency #39495How?
We're replacing a
_.kebabCase()
with its counterpart from thechange-case
library that we've been widely using.Testing Instructions
/wp-admin/edit.php?post_type=wp_block
kebab-case
).