Skip to content
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-57762: fix misleading tkinter.Tk docstring #98837

Merged
merged 1 commit into from Dec 22, 2022

Conversation

hauntsaninja
Copy link
Contributor

@hauntsaninja hauntsaninja commented Oct 29, 2022

Mentioned as a desired change by terryjreedy on the corresponding issue, since Tk is not a subclass of Toplevel.

Mentioned as a desired change by terryjreedy on the corresponding issue,
since Tk is not a subclass of Toplevel.
@hauntsaninja
Copy link
Contributor Author

hauntsaninja commented Dec 22, 2022

cc @JelleZijlstra

Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

feel free to merge

@hauntsaninja hauntsaninja merged commit ad23da0 into python:main Dec 22, 2022
13 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Dec 22, 2022

Thanks @hauntsaninja for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖

@hauntsaninja hauntsaninja deleted the gh-57762 branch Dec 22, 2022
@bedevere-bot
Copy link

bedevere-bot commented Dec 22, 2022

GH-100440 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

bedevere-bot commented Dec 22, 2022

GH-100441 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 22, 2022
Mentioned as a desired change by terryjreedy on the corresponding issue,
since Tk is not a subclass of Toplevel.
(cherry picked from commit ad23da0)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 22, 2022
Mentioned as a desired change by terryjreedy on the corresponding issue,
since Tk is not a subclass of Toplevel.
(cherry picked from commit ad23da0)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Dec 22, 2022
Mentioned as a desired change by terryjreedy on the corresponding issue,
since Tk is not a subclass of Toplevel.
(cherry picked from commit ad23da0)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Dec 22, 2022
Mentioned as a desired change by terryjreedy on the corresponding issue,
since Tk is not a subclass of Toplevel.
(cherry picked from commit ad23da0)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
@terryjreedy
Copy link
Member

terryjreedy commented Dec 22, 2022

Thank you. I somehow missed or maybe just glided over the the review request. I just verified that my comment was correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants