-
Notifications
You must be signed in to change notification settings - Fork 270
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
[Initializers] Restore sub-command (npm install / lint) console output #933
Conversation
… for stdio w/ fallback to 'pipe' (when not TTY context)
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sitecore-jss/edge-e2e-styleguide/H4yPgSDs2okZVpAdEq7WwiASaZd1 [Deployment for faac5d3 failed] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some tests are failing.
@sc-addypathania Thanks for letting me know, tests are now passing on CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
Description / Motivation
We moved away from
inherit
forstdio
in #912 to address an issue with thenpm install
hanging when run within our dotnet template for Next.js. However, this resulted in a loss of useful output. This PR goes back to usinginherit
forstdio
w/ fallback topipe
(when not TTY context).Testing Details
Tested on Node.js LTS (16.10), including within dotnet template for Next.js
Types of changes