-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
v5.0.0 has an error to include yetch-polyfill #118
Comments
@nightire I tried a fresh |
@xg-wang yes, a repro is: https://github.com/choice-form/os-client |
|
@xg-wang Hi, me again. I found this issue is back and it is becoming more complex than last time. Yesterday I was following this article to experiment with broccoli 2, and I saw this error again. I did I list the files under full error log:
|
And BTW, if I run |
Seems like the error is specifically saying that I suspect that something is trying to treat this path as a directory and this error is thrown by Broccoli 2.0 as it tries to watch directories that are not existing... |
@rwjblue yes, I can confirm it's a file. |
I suspect the issue is here: Lines 122 to 131 in 80e77c8
I'd need to dig in further, but we should probably replace the |
@nightire Can you add repro steps? I'm on your develop branch and |
@xg-wang yes, I was doing it all over again on my local machine, here is the full process and results:
and the full error report:
|
So 2 issues here.
Closing this, feel free to reopen is you still see the error after upgrading to v5.1.1 |
Upgrade to v5.0.0 recently and found there was an error:
The text was updated successfully, but these errors were encountered: