Change default behavior for output piping

[M-Zuber]

Based on #19, #23,#27 it seems that the textual add-on that npm-watch does has generated a number of issues.
I propose changing the default to simply be passing it along, and if the user explicitly opts-in, the script name could be pre-pended/output would be color stripped.
This allows us to inform the user that the opt-in modes may produce have unpleasant visual effects that are not in our control due to I/O issues.

Tagging @alxndr , @hectormenendez , @domarmstrong, @duivv for feedback as they have been involved in issues/prs related to this issue.

[leviwheatcroft]

I think this is a really important issue. Given the usecase for this package, output formatting is really important.

The current approach is to strip existing formatting (colors ?), and prefix with the script name and a date stamp. I think it's the toString() method which strips colors / formats & date stamps.

[M-Zuber]

If I understood you correctly, you are in agreement that we should switch what the default values are?

[leviwheatcroft]

Sorry, yes I'm in agreement.

[leviwheatcroft]

Any progress on this? Looking it looks like #23 was closed in favor of setting stdio to ['pipe', 'inherit', 'inherit'] but that hasn't made it in to master?

[M-Zuber]

I am so so sorry......
Thank you for reminding me
I will work on this later today

[M-Zuber]

Yeah, this it going to take another day, I want to try and get all outstanding PRs released, and then do a second version bump to make it easier for users to rollback if they need to

[mashaalmemon]

@M-Zuber , would the change from @leviwheatcroft be able to be made in master? We're experiencing log message styles/colors not making their way properly through when npm-watching the process. If inherit was set to ['pipe', 'inherit', 'inherit'] we would be all set.

Could submit a PR if it made things easier. Literally a 1 parameter change.

[M-Zuber]

Yes a PR would make things much easier, thank you very much!

[mashaalmemon]

@M-Zuber ok sounds good. We'll submit one shortly.

[mashaalmemon]

@M-Zuber , @loganwhit submitted the PR. However the build failed; looking at the error, it seems to be some issue with the CI/CD setup?

[M-Zuber]

New version is out: https://www.npmjs.com/package/npm-watch/v/0.11.0