Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

ES2015 #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ES2015 #22

wants to merge 1 commit into from

Conversation

marcobiedermann
Copy link

I've translated the syntax to ES2015. This way you can make use of the import destructure operator to only include the functions you really need. This will reduce the file size and improves the performance overall.

@okonet
Copy link
Collaborator

okonet commented Feb 17, 2018

to only include the functions you really need

No, this is not how restructuring in imports/requires works. Your change won't affect the bundle size.

Another problem with your change is that destructuring doesn't work in Node 4 so you need to upgrade / specify engines in package.json to make Node 6 an explicit dependency.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants