Skip to content

Conversation

@chris-eibl
Copy link
Member

@chris-eiblchris-eibl commented May 25, 2025

Fixes the WindowsConsoleGetEventTests which got broken due to moving wait into get_event in WindowsConsole during #133728.

@bedevere-appbedevere-appbot added awaiting review tests Tests in the Lib/test dir labels May 25, 2025
@ambvambv changed the title fix WindowsConsoleGetEventTests after PR-133728GH-130328: Fix WindowsConsoleGetEventTests after gh-133728May 25, 2025
@bedevere-appbedevere-appbot mentioned this pull request May 25, 2025
@ambv
Copy link
Contributor

ambv commented May 25, 2025

!buildbot Windows10

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ambv for commit 7f15126 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F134660%2Fmerge

The command will test the builders whose names match following regular expression: Windows10

The builders matched are:

  • AMD64 Windows10 PR

@ambv
Copy link
Contributor

ambv commented May 25, 2025

Buildbot looks happy now!

@ambvambv merged commit 1000283 into python:mainMay 25, 2025
49 checks passed
@miss-islington-app
Copy link

Thanks @chris-eibl for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 25, 2025
pythongh-134660) (cherry picked from commit 1000283) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>
@bedevere-app
Copy link

GH-134666 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14 bugs and security fixes label May 25, 2025
ambv pushed a commit that referenced this pull request May 25, 2025
…-134660) (gh-134666) (cherry picked from commit 1000283) Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>
@chris-eiblchris-eibl deleted the fix_WindowsConsoleGetEventTests branch May 26, 2025 05:55
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting reviewskip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@chris-eibl@ambv@bedevere-bot