-
Notifications
You must be signed in to change notification settings - Fork 396
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
Doesn't work with IE11 (ducks for cover) #50
Comments
I don't understand, are you shipping |
Yes, I'm using it client-side to have a pluggable framework. It works great
for that…
…On Tue, Mar 20, 2018 at 4:02 PM Florent Cailhol ***@***.***> wrote:
I don't understand, are you shipping tapable into your bundle? tapable is
just an utility used within webpack and its loaders/plugins. Could you
detail what you are trying to achieve?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#50 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AADWlo7VYIa14eMMeSJADQJKZ_w9UlINks5tgRoVgaJpZM4SEeRL>
.
|
Since the library is targeting node 6+, I don't think we would transpile it. Your best option is to include it into your |
Yes, I'm whitelisting it now. Fair enough, thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I am using this in an SPA and it's really great, but it turns out that the npm version uses
class
in the JS, so it's not ES5.Would it be possible to provide the npm code as ES5 code? I don't know what the current best way to provide ES6 code is, but this is the only package that causes problems in the build for our IE users.
In the meantime I'll try some workaround like forcing a babel compilation of this package.
The text was updated successfully, but these errors were encountered: