Skip to content

gh-111201: Allow bracketed paste to work - #118700

Merged
ambv merged 6 commits into
python:mainfrom
pablogsal:enable_bracketed_paste
May 7, 2024
Merged

gh-111201: Allow bracketed paste to work#118700
ambv merged 6 commits into
python:mainfrom
pablogsal:enable_bracketed_paste

Conversation

@pablogsal

@pablogsalpablogsal commented May 7, 2024

Copy link
Copy Markdown
Member

@bedevere-appbedevere-appBot mentioned this pull request May 7, 2024
7 tasks
@pablogsal
pablogsalforce-pushed the enable_bracketed_paste branch from e438388 to 4315668CompareMay 7, 2024 11:52
@pablogsal
pablogsal requested a review from ambvMay 7, 2024 11:56

@lysnikolaoulysnikolaou 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 couple of nitty comments. Feel free to merge.

Comment threadLib/_pyrepl/commands.py Outdated
Comment threadLib/_pyrepl/simple_interact.py Outdated
@pablogsal

Copy link
Copy Markdown
MemberAuthor

CC: @Yhg1s

ambv
ambv approved these changes May 7, 2024

@lysnikolaoulysnikolaou 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.

LGTM apart from one more nitty comment.

Comment threadLib/_pyrepl/unix_console.py Outdated
@ambv
ambv enabled auto-merge (squash) May 7, 2024 12:26
@ambv
ambv merged commit 7d90b8a into python:mainMay 7, 2024
@pablogsal
pablogsal deleted the enable_bracketed_paste branch May 7, 2024 12:55
@hauntsaninja

Copy link
Copy Markdown
Contributor

Amazing! I love living in this synthwave future

SonicField pushed a commit to SonicField/cpython that referenced this pull request May 8, 2024
cfbolz added a commit to pypy/pypy that referenced this pull request Jul 29, 2024
as far as I'm aware this should make gh-2880 go away (not 100% sure,
don't have a mac to test).
backport of parts of python/cpython#118700
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@pablogsal@hauntsaninja@ambv@lysnikolaou