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.

Sign up for GitHub

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

Jump to bottom

fix(update): change warning message stream from stdout to stderr #698

Merged
merged 3 commits into from
Nov 5, 2021
Merged

fix(update): change warning message stream from stdout to stderr #698

merged 3 commits into from
Nov 5, 2021

Conversation

sihamais
Copy link
Member

@sihamais sihamais commented Nov 4, 2021 •
edited
Loading

  • Add a changelog entry in the section "To Be Released" of CHANGELOG.md

Fix #695

@sihamais sihamais changed the title fix update warning message stream fix(warning-message-stream): change warning message stream from stdout to stderr Nov 4, 2021
@sihamais sihamais requested a review from EtienneM November 4, 2021 15:24
EtienneM
EtienneM approved these changes Nov 4, 2021
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
@sihamais @EtienneM
Update CHANGELOG.md
d91d630 
Co-authored-by: Étienne M. <EtienneM@users.noreply.github.com>
@sihamais sihamais changed the title fix(warning-message-stream): change warning message stream from stdout to stderr fix(update): change warning message stream from stdout to stderr Nov 5, 2021
@sihamais sihamais merged commit 2bbd90a into master Nov 5, 2021
@sihamais sihamais deleted the fix/695/warning-messages-stream branch November 5, 2021 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EtienneM EtienneM EtienneM approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Warning messages displayed when an upgrade is available should be displayed on stderr and not stdout
2 participants
@sihamais @EtienneM