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

Appearance settings

gh-86673: Loosen test_ttk.test_identify() requirements#125335

Merged
erlend-aasland merged 1 commit into
python:mainpython/cpython:mainfrom
erlend-aasland:fix-test-identifyerlend-aasland/cpython:fix-test-identifyCopy head branch name to clipboard
Oct 13, 2024
Merged

gh-86673: Loosen test_ttk.test_identify() requirements#125335
erlend-aasland merged 1 commit into
python:mainpython/cpython:mainfrom
erlend-aasland:fix-test-identifyerlend-aasland/cpython:fix-test-identifyCopy head branch name to clipboard

Conversation

@erlend-aasland

@erlend-aasland erlend-aasland commented Oct 11, 2024

Copy link
Copy Markdown
Contributor

In aeca373 (PR gh-12011, issue gh-71500), test_identify() was changed to expect different results on Darwin. Ned's fix was later adjusted by e52f9be. This workaround is only needed for some variants of Tk/Tcl on macOS, so we now allow both the workaround and the generic results for these tests.

In aeca373 (PR pythongh-12011, issue pythongh-71500), test_identify() was changed
to expect different results on Darwin. This workaround is only needed
for some variants of Tk/Tcl on macOS, so we now allow both the
workaround and the generic results for these tests.
@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Oct 11, 2024
@erlend-aasland erlend-aasland added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes tests Tests in the Lib/test dir skip news and removed tests Tests in the Lib/test dir awaiting core review labels Oct 11, 2024

@ned-deily ned-deily 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, thanks!

@erlend-aasland erlend-aasland merged commit 4197a79 into python:main Oct 13, 2024
@erlend-aasland erlend-aasland deleted the fix-test-identify branch October 13, 2024 07:38
@miss-islington-app

This comment has been minimized.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 13, 2024
…H-125335)

In aeca373 (PR pythongh-12011, issue pythongh-71500), test_identify() was changed to expect different results on Darwin. Ned's fix was later adjusted by e52f9be. This workaround is only needed for some variants of Tk/Tcl on macOS, so we now allow both the workaround and the generic results for these tests.
(cherry picked from commit 4197a79)

Co-authored-by: Erlend E. Aasland <erlend@python.org>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 13, 2024
…H-125335)

In aeca373 (PR pythongh-12011, issue pythongh-71500), test_identify() was changed to expect different results on Darwin. Ned's fix was later adjusted by e52f9be. This workaround is only needed for some variants of Tk/Tcl on macOS, so we now allow both the workaround and the generic results for these tests.
(cherry picked from commit 4197a79)

Co-authored-by: Erlend E. Aasland <erlend@python.org>
@bedevere-app

bedevere-app Bot commented Oct 13, 2024

Copy link
Copy Markdown

GH-125390 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Oct 13, 2024
@bedevere-app

bedevere-app Bot commented Oct 13, 2024

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Oct 13, 2024
@erlend-aasland

Copy link
Copy Markdown
Contributor Author

Thanks for the review! 🚋

erlend-aasland added a commit that referenced this pull request Oct 13, 2024
) (#125391)

In aeca373 (PR gh-12011, issue gh-71500), test_identify() was changed to expect different results on Darwin. Ned's fix was later adjusted by e52f9be. This workaround is only needed for some variants of Tk/Tcl on macOS, so we now allow both the workaround and the generic results for these tests.
(cherry picked from commit 4197a79)

Co-authored-by: Erlend E. Aasland <erlend@python.org>
erlend-aasland added a commit that referenced this pull request Oct 13, 2024
) (#125390)

In aeca373 (PR gh-12011, issue gh-71500), test_identify() was changed to expect different results on Darwin. Ned's fix was later adjusted by e52f9be. This workaround is only needed for some variants of Tk/Tcl on macOS, so we now allow both the workaround and the generic results for these tests.
(cherry picked from commit 4197a79)

Co-authored-by: Erlend E. Aasland <erlend@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.