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

Print debug warning on stderr, not stdout. #8430

Closed
wants to merge 1 commit into from

Conversation

athas
Copy link
Collaborator

@athas athas commented Aug 25, 2022

This prevents things like list-bin from being broken.


Please include the following checklist in your PR:

Please also shortly describe how you tested your change. Bonus points for added tests!

This prevents things like list-bin from being broken.
@ulysses4ever
Copy link
Collaborator

A more correct version of it has already been accepted and currently awaits a merge at #8396, so I suggest closing this. But thank you for taking the effort to fix it anyway!

@athas athas closed this Aug 25, 2022
@athas athas deleted the debug-warning-stderr branch August 25, 2022 15:50
@Mikolaj
Copy link
Member

Mikolaj commented Aug 25, 2022

@athas: if you could make a lightning review of #8396, that's always valuable.

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

Successfully merging this pull request may close these issues.

3 participants