Skip to content

Added an option to enable polyfill for Babel Transform Runtime plugin #262

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

Conversation

tzemanovic
Copy link
Contributor

Hi,

I've added an option with docs update to enable polyfill in Babel Transform Runtime plugin.

This option is recommended for libraries, while babel-polyfill is recommended for apps, but because I'm building an app that's embeddable on arbitrary websites, global polluting polyfill is not an option and so I cannot use babel-polyfill.

@halfzebra
Copy link
Owner

Thanks for this PR!

I'm trying to reduce the amount of ENV variables so unfortunately, this approach does not look very future-proof.

I will be working on a way of extending the configuration, described in #265 #280

Please let me know if it sounds like a viable solution to your problem and would make you consider using it.

@halfzebra halfzebra closed this Sep 5, 2018
@tzemanovic
Copy link
Contributor Author

I agree that this is not best approach, so I'm happy about this decision.

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.

2 participants