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
Update tmpdir
and tmpfile
context manager docstrings
#8270
Update tmpdir
and tmpfile
context manager docstrings
#8270
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for putting this together @mesejo! I left a few small suggestions, but overall this looks good
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
Co-authored-by: James Bourbeau <jrbourbeau@users.noreply.github.com>
9d4406a
to
dcc79a0
Compare
@jrbourbeau Thanks for reviewing! I agree with your suggestions and believe now it looks better. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mesejo. This looks great -- will merge after CI finishes
tmpdir
and tmpfile
context manager docstrings
tmpdir
andtmpfile
context manager docstrings #7800pre-commit run --all-files