-
Notifications
You must be signed in to change notification settings - Fork 161
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
TypeError: Cannot read property 'identifier' of undefined #91
Comments
@legomushroom I was looking at this earlier to start mulling over what is going on. This is frankly super weird. This error implies that the function erroring is getting no values when its called. No error or module. If the wrapped factory found no module, I think it is supposed to give an error which the function will forward. Is there any more of your config you can share? |
Hi @mzgoddard! Thanks for the response.
This caused the error. Yep, both |
Awesome. Thanks for letting me know that'll really help me add units and
support for Ignore.
…On Thursday, January 26, 2017, Oleg Solomka ***@***.***> wrote:
Hi @mzgoddard <https://github.com/mzgoddard>!
Thanks for the response.
I figured out the exact problem. I was using webpack.IgnorePlugin (for
moment.js issue <moment/moment#2373>) like this:
new webpack.IgnorePlugin(/^\.\/locale$/, /moment$/)
This caused the error. Yep, both error and module were undefined. Works
like a charm after removing the IgnorePlugin.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAy3EtVEPz01fQr3AUo3g4iA0OVfbyt3ks5rWS7UgaJpZM4LuNUS>
.
|
Having this error after moving to
webpack2
.Entire log:
This is the plugin's config:
Will appreciate any help!
The text was updated successfully, but these errors were encountered: