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

Include STDOUT and PIPE in public API #69

Merged
merged 1 commit into from
Dec 1, 2022

Conversation

sethfischer
Copy link
Contributor

@sethfischer sethfischer commented Dec 1, 2022

Include STDOUT and PIPE in the public API as the documentation recommends their use.

This will mean linters do not produce warnings like the following:

'STDOUT' is not declared in __all__ 

Examples from the documentation recommending the used of STDOUT and PIPE:

Demo.ipynb:

from wurlitzer import pipes, sys_pipes, STDOUT, PIPE

README.md:

from wurlitzer import pipes, STDOUT

Include STDOUT and PIPE in the public API as the documentation
recommends their use.
@minrk minrk merged commit 7c433d2 into minrk:main Dec 1, 2022
@minrk
Copy link
Owner

minrk commented Dec 1, 2022

Thanks!

@minrk
Copy link
Owner

minrk commented Dec 1, 2022

released 3.0.3 with this

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