Skip to content

[3.10] gh-92311: Let frame_setlineno jump over listcomps - #92717

Merged
markshannon merged 5 commits into
python:3.10from
sweeneyde:setlineno_comprehension
May 12, 2022
Merged

markshannon merged 5 commits into
python:3.10from
sweeneyde:setlineno_comprehension

Conversation

@sweeneyde

Copy link
Copy Markdown
Member

No description provided.

@markshannon markshannon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few nits, but otherwise looks good.
Thanks for fixing this.

Comment thread Lib/test/test_sys_settrace.py Outdated
Comment thread Lib/test/test_sys_settrace.py Outdated
Comment thread Lib/test/test_sys_settrace.py Outdated
Comment thread Lib/test/test_sys_settrace.py Outdated
@bedevere-bot

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@sweeneyde

Copy link
Copy Markdown
Member Author

I have made the requested changes; please review again

@bedevere-bot

Copy link
Copy Markdown

Thanks for making the requested changes!

@markshannon: please review the changes made to this pull request.

@bedevere-bot
bedevere-bot requested a review from markshannon May 12, 2022 10:30
@markshannon

Copy link
Copy Markdown
Member

Thanks.

@markshannon
markshannon merged commit a24e676 into python:3.10 May 12, 2022
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @sweeneyde for the PR, and @markshannon for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington

Copy link
Copy Markdown
Contributor

Sorry @sweeneyde and @markshannon, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker a24e67697362e76dd25d6901109277458b5971b9 3.9

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.

5 participants