Skip to content
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

2.1 -> 3.0 upgrade surprise: stream: process.stderr change #85

Closed
izelnakri opened this issue Aug 5, 2018 · 1 comment
Closed

2.1 -> 3.0 upgrade surprise: stream: process.stderr change #85

izelnakri opened this issue Aug 5, 2018 · 1 comment

Comments

@izelnakri
Copy link

izelnakri commented Aug 5, 2018

What is the reasoning behind changing the default stream from stdout to stderr?

It took me quite sometime to figure out what broke in my tests. I understand it is a major version change but would be nice if it was in the CHANGELOG/README.

Wouldnt it be better to keep stdout as default as more people are having the same issue? #61

@izelnakri izelnakri changed the title 2.1 -> 3.0 upgrade suprise: stream: process.stderr change 2.1 -> 3.0 upgrade surprise: stream: process.stderr change Aug 5, 2018
@sindresorhus
Copy link
Owner

sindresorhus commented Aug 6, 2018

The default stream did not change in 2.1=>3.0: v2.1.0...v3.0.0

It has always been stderr: 3467596#diff-168726dbe96b3ce427e7fedce31bb0bcR11

The reason it's stderr is so that it will not interfere with normal output. Most other spinners use stderr too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants