-
Notifications
You must be signed in to change notification settings - Fork 12.8k
Debug failure at walk #50675
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
Comments
Do you have a code sample for this? What version of TS are you on? The line number you have doesn't line up with 4.8.2. |
|
I don't have a code sample since this is just running |
Agh, I got the line numbers wrong. The code in question is: TypeScript/src/compiler/program.ts Lines 2378 to 2379 in fd3a84c
Can you run |
Unfortunately the error happens as well when
|
Is this repo accessible anywhere for testing? It's fine if the "code sample" is the whole repo, it's just hard to diagnose this without anything to go on. |
I am having the same exact issue. Tried with 4.7.4 and 4.8.3
Also tried to listFiles etc, I am getting the same issue |
Removing |
After a bit further digging, turning allowJs from true to false made the issue go away, but its not a fix though |
Yes, definitely, but what I need is to find that file. |
@jakebailey any other idea how I could help you to find the issue? |
If you can't provide the repo, the only thing I can suggest would be to try and exclude JS files until the offending one is found and it no longer crashes. |
Removing the
Full command output collapsed for readability
However, this error now is coming up in the beginning. I don't have any directories with this name on my project.
|
Are there any .js files with any of those names in the error message? |
For me the problem was coming from the js files in ./dist/bundles folder that was generated after running |
If you are able to identify a specific piece of code, that would be great. Even though it's definitely a configuration error to not ignore an output directory like |
I believe the problem is that we have .js files, but after switching to the hermes engine they contain bytecode instead of javascript. If I switch back to the jsc engine it will bundle to javascript and won't get the error. |
|
Bug Report
Running
tsc
causes an error message:🔎 Search Terms
Debug failure
🕗 Version & Regression Information
💻 Code
🙁 Actual behavior
Running
tsc
gives aDebug failure
error.🙂 Expected behavior
Running
tsc
should have worked without an errorThe text was updated successfully, but these errors were encountered: