Skip to content

Include the Babel polyfill #58

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

Merged
merged 1 commit into from
Jun 7, 2017
Merged

Include the Babel polyfill #58

merged 1 commit into from
Jun 7, 2017

Conversation

s100
Copy link
Contributor

@s100 s100 commented Jun 7, 2017

IE 11 and PhantomJS do not support ES6 features such as Symbol and it looks like I didn't configure .babelrc to pull in the polyfill for this feature.

Unfortunately this bloats jsonata-es5.js even further, from 212kB to 277kB. jsonata-es5.min.js now increases from 53kB to 85kB. So this is just a temporary instant fix for now, until I figure out how to pull in only the Symbol polyfill by itself and nothing else.

@coveralls
Copy link

coveralls commented Jun 7, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling cfa59a9 on s100:add-polyfill into 7ccf1f9 on jsonata-js:master.

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.

3 participants