-
Notifications
You must be signed in to change notification settings - Fork 45
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
"File does not exist" is confusing #19
Comments
I could probably make the plugin detect when this occurs and completely stop attempting to enforce anything, since presumably your build is about to fail anyway. Would that work? |
Yes, thanks! |
I reverted to 1.1.4 which doesn't have this issue (or at least doesn't reproduce as often).
In 1.1.4, it compiles successfully after the file is created, but not in 2.0.0. |
I'm going to try and look at these issues this week. Sorry for taking so long, but I switched jobs and don't actually use this code anymore. I'll try to help out for those who still do though :-) |
No worries 😛 |
I think I have it now, I'm still working on hardening tests. One is still
intermittently failing for no good rain.
…On Tue, May 30, 2017, 10:36 cesare soldini ***@***.***> wrote:
I think this may actually be a duplicate of #17
<#17>
😄
I think it could be enough to juste delete the FS caching code, as
webpack2 has a special fs module with integrated caching (as far as I
understood, please any webpack expert verify this).
I could take a look if @Urthen doesn't find the time!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#19 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAb_dqRgNM0LcJFYKgNH63u1B7nbmeJvks5r_DeOgaJpZM4NeRzB>
.
|
Published 2.1.0 which as you mentioned should fix both this and #17. Can you give it a test? |
Scratch that, try with 2.1.1 -- .0 had problems with older versions of node. |
When I switch between branches, I often see messages like this:
Note how it starts with the case sensitive plugin intro, but then the real issue is that the file just stopped existing. So the message is misleading until the very last four words.
Shouldn't the plugin just bail out if a file was deleted, and ignore it? Then webpack can show its normal message about file missing.
The text was updated successfully, but these errors were encountered: