Skip to content

Conversation

@csabella
Copy link
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

@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
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

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