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-79856: IDLE - separate config_key window and frame #11427

Merged
merged 2 commits into from Sep 30, 2022

Conversation

csabella
Copy link
Contributor

@csabella csabella commented Jan 3, 2019

Split dialog class into a Window class and Frame class and adjust tests to match.

@csabella csabella requested a review from terryjreedy as a code owner Jan 3, 2019
@terryjreedy terryjreedy changed the title bpo-35598: IDLE: Refactor configkey.py window and frame class bpo-35675: IDLE config_key - separate window and frame classes Jan 7, 2019
Copy link

@auvipy auvipy left a comment

rebase

iritkatriel
iritkatriel previously requested changes Sep 30, 2022
Copy link
Member

@iritkatriel iritkatriel left a comment

Needs to be rebased.

@bedevere-bot
Copy link

bedevere-bot commented Sep 30, 2022

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@iritkatriel iritkatriel added the pending The issue will be closed if no feedback is provided label Sep 30, 2022
@terryjreedy terryjreedy changed the title bpo-35675: IDLE config_key - separate window and frame classes bpo-35675: IDLE config_key - separate window and frame Sep 30, 2022
@terryjreedy terryjreedy removed awaiting changes pending The issue will be closed if no feedback is provided labels Sep 30, 2022
@terryjreedy terryjreedy self-assigned this Sep 30, 2022
@terryjreedy terryjreedy changed the title bpo-35675: IDLE config_key - separate window and frame bpo-35675: IDLE - separate config_key window and frame Sep 30, 2022
@terryjreedy terryjreedy dismissed iritkatriel’s stale review Sep 30, 2022

Conflick resolved

@terryjreedy terryjreedy merged commit 1cc308d into python:main Sep 30, 2022
12 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Sep 30, 2022

Thanks @csabella for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

bedevere-bot commented Sep 30, 2022

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

@bedevere-bot
Copy link

bedevere-bot commented Sep 30, 2022

GH-97684 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 Sep 30, 2022
bpo-35598: IDLE: Refactor window and frame class

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 1cc308d)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 30, 2022
bpo-35598: IDLE: Refactor window and frame class

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 1cc308d)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
miss-islington added a commit that referenced this pull request Sep 30, 2022
bpo-35598: IDLE: Refactor window and frame class

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 1cc308d)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
miss-islington added a commit that referenced this pull request Sep 30, 2022
bpo-35598: IDLE: Refactor window and frame class

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 1cc308d)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
@terryjreedy terryjreedy changed the title bpo-35675: IDLE - separate config_key window and frame gh-79856: IDLE - separate config_key window and frame Sep 30, 2022
serhiy-storchaka pushed a commit to serhiy-storchaka/cpython that referenced this pull request Oct 2, 2022
bpo-35598: IDLE: Refactor window and frame class

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
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

8 participants