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 warning messages to stderr #58

Merged
merged 1 commit into from
Aug 20, 2021

Conversation

pbderr
Copy link
Contributor

@pbderr pbderr commented Aug 19, 2021

Modify the internal_warn function so that it prints to stderr instead of stdout so that the CSV output is not corrupted with non-CSV content. This would solve issue #57

@david-a-wheeler
Copy link
Owner

Oy! You're absolutely right! Sorry about that.

@david-a-wheeler david-a-wheeler merged commit d9ddc06 into david-a-wheeler:master Aug 20, 2021
@pbderr
Copy link
Contributor Author

pbderr commented Sep 2, 2021 via email

@david-a-wheeler
Copy link
Owner

Already done! 2.0.19 has been released & specifically includes this change.

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.

None yet

2 participants