Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[3.10] bpo-45624: make test_graphlib not depend on the iteration order of sets (GH-29233)#29293

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ambv merged 1 commit intopython:3.10frommiss-islington:backport-7401694-3.10
Oct 28, 2021

Conversation

@miss-islington
Copy link
Contributor

@miss-islingtonmiss-islington commentedOct 28, 2021
edited by bedevere-bot
Loading

the current test depended on integer sets being iterated on in a certain
fixed order. That order is different on PyPy (insertion based) and could
change in CPython in the future in theory. Make the test robust against
a different iteration order by sorting.
(cherry picked from commit7401694)

Co-authored-by: Carl Friedrich Bolz-Tereickcfbolz@gmx.de

https://bugs.python.org/issue45624

…ts (pythonGH-29233)the current test depended on integer sets being iterated on in a certainfixed order. That order is different on PyPy (insertion based) and couldchange in CPython in the future in theory. Make the test robust againsta different iteration order by sorting.(cherry picked from commit7401694)Co-authored-by: Carl Friedrich Bolz-Tereick <cfbolz@gmx.de>
@miss-islington
Copy link
ContributorAuthor

@cfbolz and@ambv: Status check is done, and it's a success ✅ .

@ambvambv merged commiteccb753 intopython:3.10Oct 28, 2021
@miss-islingtonmiss-islington deleted the backport-7401694-3.10 branchOctober 28, 2021 21:15
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@miss-islington@ambv@the-knights-who-say-ni@bedevere-bot@cfbolz

[8]ページ先頭

©2009-2025 Movatter.jp