-
Notifications
You must be signed in to change notification settings - Fork 286
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
Clarification on non-browserify integration #128
Comments
I haven't tried it with webpack, but it does not work with rollup. |
I think webpack should work, that's how we use(d) at the FT. |
[didn't mean to close this, sorry!] |
I'm using webpack and it doesnt work on old browsers that don't have fetch variable. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Given the description of this project:
It seems to imply it is incompatible with webpack. However, looking at the code here, there's this comment:
// Return that as the export for use in Webpack, Browserify etc.
...and I don't see anything that is explicitly Browserify-only here.
Has this been verified to work/not work with webpack bundles? It would be nice to call this out explicitly in the README.
The text was updated successfully, but these errors were encountered: