Skip to content

bpo-30579: Docs for dynamic traceback creation #5653

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

Merged
merged 1 commit into from
Feb 13, 2018

Conversation

ncoghlan
Copy link
Contributor

@ncoghlan ncoghlan commented Feb 13, 2018

@ncoghlan
Copy link
Contributor Author

I merged this without doc updates prior to beta 1, so this backfills the missing details.

The handling of frame and traceback objects in the language reference could stand to be significantly improved in general, but this is a deliberately minimal update in order to cover the changes that were made.

@ncoghlan ncoghlan merged commit aec7532 into python:master Feb 13, 2018
@miss-islington
Copy link
Contributor

Thanks @ncoghlan for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 13, 2018
(cherry picked from commit aec7532)

Co-authored-by: Nick Coghlan <[email protected]>
@bedevere-bot
Copy link

GH-5655 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit that referenced this pull request Feb 13, 2018
(cherry picked from commit aec7532)

Co-authored-by: Nick Coghlan <[email protected]>
@ncoghlan ncoghlan deleted the whats-new-mutable-tracebacks branch March 30, 2018 07:53
@@ -198,16 +198,23 @@ Standard names are defined for the following types:
Defaults to ``None``. Previously the attribute was optional.


.. data:: TracebackType
.. class:: TracebackType(tb_next, tb_frame, tb_lasti, tb_lineno)
Copy link
Member

@merwok merwok Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In class directives, things in parens should be base classes, not constructor parameters.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@merwok This PR is old enough now that fixing the markup here is likely better handled as a separate issue on the tracker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants