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
fix(js): return success false if tsc compilation fails #9438
fix(js): return success false if tsc compilation fails #9438
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/GSGpduWMpVxaRQdTStydvK23odE8 |
@AgentEnder should there be some sort of test to prevent recurrence? |
Yeah, we'll be looking into some kind of testing for this. At the unit test level its not an easy thing to do, but some follow up PR can include an e2e test for similar scenarios. In particular, this one only fails if on Node 14 or below, since unhandled promise rejections don't propagate errors up. |
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
compileTypescript
throws an error if compilation fails. This causes Node 14 + 12 to hang if compileTypescript is invoked by an async function, since unhandled promise rejections do not cause the process to fail.Expected Behavior
compileTypescript
reportssuccess: false
on build failures.Related Issue(s)
Fixes #9046
Fixes #9251