Skip to content

ES5 Support#8

Open
alan2207 wants to merge 2 commits into
VictorTaelin:masterfrom
alan2207:master
Open

ES5 Support#8
alan2207 wants to merge 2 commits into
VictorTaelin:masterfrom
alan2207:master

Conversation

@alan2207

@alan2207 alan2207 commented Jul 17, 2018

Copy link
Copy Markdown

Added .babelrc file to define babel-preset-es2015 in presets.
Transpiled the lib folder to ES5 so that it is supported by all browsers.

The reason behind this pull request was this error caused by non-transpiled ES6 code in the current lib folder.

@godfreyhobbs

godfreyhobbs commented Jul 27, 2018

Copy link
Copy Markdown

Our project needs this. We get this error with create-react-app build:

Creating an optimized production build...
Failed to compile.

Failed to minify the code from this file: 

 	./node_modules/eth-crypto/node_modules/eth-lib/lib/account.js:6 

@lantelyes

lantelyes commented Apr 12, 2019

Copy link
Copy Markdown

Can we merge this soon? Our project needs it as well

@alan2207

alan2207 commented Apr 13, 2019

Copy link
Copy Markdown
Author

@lantelyes if you can migrate to using CRA v2 there is no need for transpiling to ES5 and this issue will not happen.

https://facebook.github.io/create-react-app/docs/troubleshooting#npm-run-build-fails-to-minify

@subey

subey commented Jun 18, 2019

Copy link
Copy Markdown

Any chance for merge?

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