Skip to content

bpo-30779: IDLE: fix delete_section calls in configdialog#2667

Merged
terryjreedy merged 2 commits intopython:masterfrom
csabella:bpo30779
Jul 11, 2017
Merged

bpo-30779: IDLE: fix delete_section calls in configdialog#2667
terryjreedy merged 2 commits intopython:masterfrom
csabella:bpo30779

Conversation

@csabella
Copy link
Copy Markdown
Contributor

changes.delete_section requires 2 parameters, so I was receiving an error message when trying to delete a custom theme.

@mention-bot
Copy link
Copy Markdown

@csabella, thanks for your PR! By analyzing the history of the files in this pull request, we identified @terryjreedy, @kbkaiser and @mlouielu to be potential reviewers.

@csabella csabella changed the title bpo30779: IDLE: fix for delete section in configdialog bpo-30779: IDLE: fix for delete section in configdialog Jul 11, 2017
@terryjreedy terryjreedy changed the title bpo-30779: IDLE: fix for delete section in configdialog bpo-30779: IDLE: fix delete_section calls in configdialog Jul 11, 2017
@terryjreedy terryjreedy merged commit 6d13b22 into python:master Jul 11, 2017
@terryjreedy
Copy link
Copy Markdown
Member

Original error was in commit f22fc6e of PR 2612.

terryjreedy pushed a commit to terryjreedy/cpython that referenced this pull request Jul 11, 2017
…og (pythonGH-2667)

Also improve test of config.ConfigChanges.delete_section.
Original patch by Cheryl Sabella.
(cherry picked from commit 6d13b22)
@bedevere-bot
Copy link
Copy Markdown

GH-2674 is a backport of this pull request to the 3.6 branch.

terryjreedy added a commit that referenced this pull request Jul 11, 2017
…og (GH-2667) (#2674)

Also improve test of config.ConfigChanges.delete_section.
Original patch by Cheryl Sabella.
(cherry picked from commit 6d13b22)
@csabella csabella deleted the bpo30779 branch August 28, 2017 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants