Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

send json errors to stderr, not stdout #2155

Closed
wants to merge 1 commit into from
Closed

send json errors to stderr, not stdout #2155

wants to merge 1 commit into from

Conversation

isaacs
Copy link
Contributor

@isaacs isaacs commented Nov 11, 2020

Fixes: #2150

References

@isaacs isaacs requested a review from a team as a code owner November 11, 2020 01:16
@ruyadorno ruyadorno added Release 7.x work is associated with a specific npm 7 release release: next These items should be addressed in the next release labels Nov 11, 2020
@raineorshine raineorshine mentioned this pull request Nov 13, 2020
Closed
3 tasks
darcyclarke pushed a commit that referenced this pull request Nov 13, 2020
@isaacs @darcyclarke
send json errors to stderr, not stdout
6d8d3cc 
Fixes: #2150
Credit: @isaacs
Close: #2155
Reviewed-by: @ruyadorno
@isaacs isaacs closed this in 1dbf0f9 Nov 13, 2020
@darcyclarke darcyclarke mentioned this pull request Nov 13, 2020
Closed
@lukekarrys lukekarrys mentioned this pull request Oct 7, 2021
Closed
@nlf nlf deleted the isaacs/json-errors-to-stderr branch March 28, 2022 16:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ljharb ljharb ljharb approved these changes

@ruyadorno ruyadorno ruyadorno approved these changes

Assignees
No one assigned
Labels
release: next These items should be addressed in the next release Release 7.x work is associated with a specific npm 7 release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] --json outputs errors to stdout instead of stderr
3 participants
@isaacs @ljharb @ruyadorno