FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-106584: Fix exit code for unittest in Python 3.12 (GH-106588) by miss-islington · Pull Request #109725 · python/cpython · GitHub

Repository navigation

[3.12] gh-106584: Fix exit code for unittest in Python 3.12 (GH-106588) - #109725

Merged
Yhg1s merged 1 commit into
python:3.12from
miss-islington:backport-8fc0713-3.12
Oct 2, 2023
Merged

Yhg1s merged 1 commit into
python:3.12from
miss-islington:backport-8fc0713-3.12

Conversation

miss-islington commented Sep 22, 2023 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

(cherry picked from commit 8fc0713)

Co-authored-by: EliseevEgor egor.eliseev@jetbrains.com
Co-authored-by: sunmy2019 59365878+sunmy2019@users.noreply.github.com
Co-authored-by: Nikita Sobolev mail@sobolevn.me

…106588)

(cherry picked from commit 8fc0713)

Co-authored-by: EliseevEgor <egor.eliseev@jetbrains.com>
Co-authored-by: sunmy2019 <59365878+sunmy2019@users.noreply.github.com>
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>

ambv commented Sep 22, 2023

Copy link
Copy Markdown
Contributor

This is a bug in 3.12.0, the fix should land for 3.12.1.

Yhg1s merged commit 551aa6a into python:3.12 Oct 2, 2023
miss-islington deleted the backport-8fc0713-3.12 branch October 2, 2023 15:18

sodul commented Dec 12, 2023

Copy link
Copy Markdown

@miss-islington, @ambv I suspect this change is breaking green on skipped tests: CleanCut/green#277

Green is tracking test duration by setting a time when startTest is called then calculates the duration in stopTest, unfortunately the changes here are now skipping startTest but not stopTest, if I read the changes correctly.

I will work with the green maintainer to update green to be compatible. There is no issue if no tests are skipped.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL