-
-
Notifications
You must be signed in to change notification settings - Fork 30k
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
bpo-33809: add the TracebackException.print() method #24231
Conversation
This PR is stale because it has been open for 30 days with no activity. |
🤖 New build scheduled with the buildbot fleet by @iritkatriel for commit 4abf619 🤖 If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again. |
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.
Approved after adding default to doc. And when you do, no re-review needed ;-).
The PR factors print code out of print_exception and moves it to the class so it can be used with any class instance, however created. I agree with this solution to the bpo request.
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
Thanks @terryjreedy ! |
I have made the requested changes; please review again. |
Thanks for making the requested changes! @terryjreedy: please review the changes made to this pull request. |
I know you said no need for re-review, but I'm trying to keep the bot happy. |
|
|
Move the content of the module-level traceback.print_exception() into a new print() method on TracebackException so that people can do
TracebackException(exc, various configuration options ... ).print()
https://bugs.python.org/issue33809