Skip to content

gh-128481: indicate that the default value for FrameSummary.end_lineno changed in 3.13 #130755

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

Conversation

picnixz
Copy link
Member

@picnixz picnixz commented Mar 2, 2025

Passing end_lineno=None to FrameSummary in Python 3.13 changed compared to Python 3.12.

3.14:

self.end_lineno = lineno if end_lineno is None else end_lineno

3.13:

self.end_lineno = lineno if end_lineno is None else end_lineno

3.12:

self.end_lineno = end_lineno


📚 Documentation preview 📚: https://cpython-previews--130755.org.readthedocs.build/

@picnixz
Copy link
Member Author

picnixz commented Mar 2, 2025

FTR, this was changed in 939fc6d. I'll merge this one before eating this evening.

@picnixz picnixz self-assigned this Mar 2, 2025
@picnixz picnixz merged commit c6513f7 into python:main Mar 2, 2025
33 checks passed
@miss-islington-app
Copy link

Thanks @picnixz for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@github-project-automation github-project-automation bot moved this from Todo to Done in Docs PRs Mar 2, 2025
@picnixz picnixz deleted the doc/frame-summary/version-changed-312-313-128481 branch March 2, 2025 17:16
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 2, 2025
…d_lineno` changed in 3.13 (pythonGH-130755)

The value taken by `FrameSummary.end_lineno` when passing `end_lineno=None` changed in pythongh-112097.

Previously, a `end_lineno` could be specified to be `None` directly but since 939fc6d, passing None makes
the constructor use the value of `lineno` instead.
(cherry picked from commit c6513f7)

Co-authored-by: Bénédikt Tran <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 2, 2025

GH-130767 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Mar 2, 2025
picnixz added a commit that referenced this pull request Mar 2, 2025
…nd_lineno` changed in 3.13 (GH-130755) (#130767)

gh-128481: indicate that the default value for `FrameSummary.end_lineno` changed in 3.13 (GH-130755)

The value taken by `FrameSummary.end_lineno` when passing `end_lineno=None` changed in gh-112097.

Previously, a `end_lineno` could be specified to be `None` directly but since 939fc6d, passing None makes
the constructor use the value of `lineno` instead.
(cherry picked from commit c6513f7)

Co-authored-by: Bénédikt Tran <[email protected]>
seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
…d_lineno` changed in 3.13 (python#130755)

The value taken by `FrameSummary.end_lineno` when passing `end_lineno=None` changed in pythongh-112097.

Previously, a `end_lineno` could be specified to be `None` directly but since 939fc6d, passing None makes
the constructor use the value of `lineno` instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants