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

Babel #1064

Closed
wants to merge 1 commit into from
Closed

Babel #1064

wants to merge 1 commit into from

Conversation

louisgv
Copy link

@louisgv louisgv commented Nov 8, 2017

This will resolve #938 , multiformats/js-cid#38 and multiformats/js-cid#42

TODO:

Add this to the Travis pipeline and expose it as the final build for other module to import.

@louisgv
Copy link
Author

louisgv commented Nov 8, 2017

Actually the built lib generates this error:

TypeError: 'caller', 'callee', and 'arguments' properties may not be accessed on strict mode functions or the arguments objects for calls to them

@@ -19,6 +19,7 @@
"npm": ">=3.0.0"
},
"scripts": {
"babel": "babel dist -d lib",
Copy link
Member

Choose a reason for hiding this comment

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

Could this create a a file to dist/index.es5.js instead?

@daviddias daviddias changed the title Adding babel Babel Feb 15, 2018
@daviddias
Copy link
Member

Just want to point out that @fazo96 provided some good intel on how to work around the fact that uglify2 doesn't support ES6, check it here #938 (comment)

@daviddias
Copy link
Member

Closing this one, see for more details. #1321

@daviddias daviddias closed this Apr 24, 2018
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.

Does not work with UglifyJS version 2
2 participants