8000 Update test.support.interpreters to include missing RunFailedError import by BruceEckel · Pull Request #103841 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

Update test.support.interpreters to include missing RunFailedError import #103841

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 26, 2023
Merged

Update test.support.interpreters to include missing RunFailedError import #103841

merged 3 commits into from
Jun 26, 2023

Conversation

BruceEckel
Copy link
Contributor

Missing RunFailedError import (Eric Snow coached me to do this)

@BruceEckel
Update interpreters.py
a379f77 
Missing RunFailedError import
@ghost
Copy link
ghost commented Apr 25, 2023 •
edited by ghost
Loading

All commit authors signed the Contributor License Agreement.
CLA signed

@ambv
Copy link
Contributor
ambv commented Apr 25, 2023

@BruceEckel CLA signed successfully, thanks. I'm increasing capacity to survive the sprint peak. Sorry for the instability!

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@BruceEckel
Copy link
Contributor Author

@ericsnowcurrently this is the one you coached me through. Do I need a NEWS entry? That is unfamiliar to me. but I think I got through everything else. First time, not sure.

@ericsnowcurrently
Copy link
Member

Thanks for working on this @BruceEckel! Feel free to add a NEWS entry. I wouldn't add one personally, but it's an opportunity for you to get visible attribution for your contribution.

@ambv ambv added needs backport to 3.12 only security fixes tests Tests in the Lib/test dir skip issue skip news labels Jun 26, 2023
@ambv ambv changed the title Update interpreters.py to include missing RunFailedError import Update test.support.interpreters to include missing RunFailedError import Jun 26, 2023
@ambv ambv merged commit 1a2bc94 into python:main Jun 26, 2023
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

GH-106110 is a backport of this pull request to the 3.12 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 26, 2023
@BruceEckel @AlexWaygood @miss-islington
Update test.support.interpreters to include missing RunFailedError im…
ecaf422 
…port (pythonGH-103841)

(cherry picked from commit 1a2bc94)

Co-authored-by: Bruce Eckel <BruceTEckel@gmail.com>
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label Jun 26, 2023
ambv pushed a commit that referenced this pull request Jun 26, 2023
@miss-islington @BruceEckel @AlexWaygood
[3.12] Update test.support.interpreters to include missing RunFailedE…
e923971 
…rror import (GH-103841) (#106110)

(cherry picked from commit 1a2bc94)

Co-authored-by: Bruce Eckel <BruceTEckel@gmail.com>
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
skip issue skip news tests Tests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@BruceEckel @ambv @bedevere-bot @ericsnowcurrently @miss-islington @AlexWaygood
0