-
-
Notifications
You must be signed in to change notification settings - Fork 30.8k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gh-104855: Update more tk tests for 8.7 #105047
Conversation
Where applicable: * Add '' to the valid options in error messages. * Test that '' is valid. * Test that 'to' < 'from' is valid instead of invalid. This PR also fixes pythongh-104856.
As before, I will wait for Christopher to test on 8.7 and Serhiy to comment and possibly revise. |
@serhiy-storchaka Please review |
@terryjreedy, there were errors in these fixes, and there were also many other test failures (and even crashes, due to Tk bugs). I fixed them all (at least on Linux), and think that this is ready to merge. If there are Windows or macOS specific failures, they can be fixed in a separate PR. |
1a6276a
to
dea00ba
Compare
59e0b40
to
358bb74
Compare
I expected to only fix few errors in this PR by committing in the original branch, but it required much more changes. So I created a branch in my repository and a new PR #120824. |
Where applicable:
This PR also addresses gh-104856 .