Skip to content

bpo-23205: IDLE: Add tests and refactor grep's findfiles#12203

Merged
csabella merged 7 commits into
python:masterfrom
csabella:bpo23205
Mar 23, 2019
Merged

bpo-23205: IDLE: Add tests and refactor grep's findfiles#12203
csabella merged 7 commits into
python:masterfrom
csabella:bpo23205

Conversation

@csabella

@csabella csabella commented Mar 6, 2019

Copy link
Copy Markdown
Contributor
  • Add tests for grep's findfiles.
  • Move findfiles from inside the GrepDialog class to be a module function.
  • Refactor findfiles to use os.walk.

Based on a patch by Al Sweigart.

https://bugs.python.org/issue23205

@terryjreedy terryjreedy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Comment thread Lib/idlelib/grep.py Outdated
@csabella
csabella merged commit d60f658 into python:master Mar 23, 2019
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @csabella for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @csabella for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-12512 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 23, 2019
)

* Add tests for grep findfiles.
* Move findfiles to module function.
* Change findfiles to use os.walk.

Based on a patch by Al Sweigart.
(cherry picked from commit d60f658)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
miss-islington added a commit that referenced this pull request Mar 23, 2019
* Add tests for grep findfiles.
* Move findfiles to module function.
* Change findfiles to use os.walk.

Based on a patch by Al Sweigart.
(cherry picked from commit d60f658)

Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
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.

6 participants