Skip to content

Conversation

@mgmacias95
Copy link
Contributor

@mgmacias95 mgmacias95 commented Jul 15, 2024

This PR fixes the behaviour when sending a KeyboardInterrupt signal to the reader. I was unable to reproduce this behaviour on unit tests so I had to test it manually. I'm attaching a GIF of the tests I made to make the review easier:
out

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

Confirmed that this fixes #121295

Comment on lines +1 to +2
Fix PyREPL console getting into a blocked state after interrupting a long
paste
Copy link
Member

Choose a reason for hiding this comment

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

Tiny wording nitpick:

Suggested change
Fix PyREPL console getting into a blocked state after interrupting a long
paste
Fixed REPL getting into a blocked state after interruption of
paste.

@ambv ambv added topic-repl Related to the interactive shell needs backport to 3.13 bugs and security fixes labels Jul 15, 2024
@ambv ambv merged commit 498a94c into python:main Jul 15, 2024
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 15, 2024
@bedevere-app
Copy link

bedevere-app bot commented Jul 15, 2024

GH-121826 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 Jul 15, 2024
@ambv
Copy link
Contributor

ambv commented Jul 15, 2024

Thanks, Marta! ✨ 🍰 ✨

ambv pushed a commit that referenced this pull request Jul 16, 2024
@mgmacias95 mgmacias95 deleted the fix_long_paste_sigint branch July 16, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic-repl Related to the interactive shell

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants