-
Notifications
You must be signed in to change notification settings - Fork 39
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
Problem using with CRA 1.0 #35
Comments
I hadn't noticed this bug when I started down this rabbit hole myself. See mishoo/UglifyJS#2011 for a long parallel path. TLDR: it seems this (https://github.com/substack/webworkify/blob/master/index.js#L63) code is violating some assumption that @stereobooster note: CRA 1.0.7 has this problem worked around, but it seems to me there's still an incompatibility between |
|
Great find, thank you @kzc! Looks like there's nothing we can do on the webworkify side to fix the issue then. Hopefully there will be a solution upstream. |
CRA stands for create-react-app. This repo is
browserify
plugin. You probably wonder how I get here with CRA, which in turn uses Webpack. Here is the story:I'm trying to make react-mapbox-gl work with CRA. react-mapbox-gl is react component for mapbox-gl, which in turn use browserify and webworkify. I'm not sure if this problem of webworkify or not. I suppose because I see blobs and workers in stack trace. Tell me if this CRA/Webpack/Mapbox error.
Uncaught ReferenceError: e is not defined
Related: alex3165/react-mapbox-gl#231
Note: prior CRA 1, for example 0.9.5, it worked fine.
To reproduce problem:
The text was updated successfully, but these errors were encountered: