-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[now-cli] Spawn builder child processes with stdio: 'inherit'
in now dev
#3113
Conversation
…ow dev` Inherit the `now dev` process stdio streams in builder child processes, so that ANSI color codes may be used when stdout is a TTY.
Need to look into why this is causing the unit tests to hang. Surely has to do with builder child processes not exiting correctly / completely. |
Explicitly send the SIGINT / SIGTERM signal to `now dev` server child processes, so that they are not left running when running the now-dev unit tests. Related to #3113 which has hanging unit tests that never "complete".
Explicitly send the SIGINT / SIGTERM signal to `now dev` server child processes, so that they are not left running when running the now-dev unit tests. Related to #3113 which has hanging unit tests that never "complete".
Explicitly send the SIGINT / SIGTERM signal to `now dev` server child processes, so that they are not left running when running the now-dev unit tests. Related to #3113 which has hanging unit tests that never "complete".
Codecov Report
@@ Coverage Diff @@
## canary #3113 +/- ##
==========================================
+ Coverage 12.29% 12.32% +0.02%
==========================================
Files 268 268
Lines 10204 10196 -8
Branches 1279 1275 -4
==========================================
+ Hits 1255 1257 +2
+ Misses 8859 8852 -7
+ Partials 90 87 -3
Continue to review full report at Codecov.
|
packages/now-cli/test/fixtures/unit/now-dev-static-build-routing/now.json
Show resolved
Hide resolved
Explicitly send the SIGINT / SIGTERM signal to `now dev` server child processes, so that they are not left running when running the now-dev unit tests. Related to #3113 which has hanging unit tests that never "complete".
…ow dev` (#3113) Inherit the `now dev` process stdio streams in builder child processes, so that ANSI color codes may be used when stdout is a TTY. **Examples:** _Next.js_ <img width="523" alt="Screen Shot 2019-10-01 at 3 42 33 PM" src="https://user-images.githubusercontent.com/71256/66006087-4fe75780-e462-11e9-927f-1e81466c4108.png"> _Gatsby_ (depends on #3112) <img width="507" alt="Screen Shot 2019-10-01 at 3 40 02 PM" src="https://user-images.githubusercontent.com/71256/66006094-5d9cdd00-e462-11e9-81da-e60bd9516778.png"> Fixes #3135.
Inherit the
now dev
process stdio streams in builder child processes, so that ANSI color codes may be used when stdout is a TTY.Examples:
Next.js
Gatsby (depends on #3112)
Fixes #3135.