[issue23216] IDLE grep/find/replace source code needs docstrings

Change by Cheryl Sabella : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ __

[issue23216] IDLE grep/find/replace source code needs docstrings

miss-islington added the comment: New changeset b34f1aa81433d60aee7bd744352b347dd650ca84 by Miss Islington (bot) in branch '3.7': bpo-23216: IDLE: Add docstrings to search modules (GH-12141) https://github.com/python/cpython/commit/b34f1aa81433d60aee7bd744352b347dd650ca84 -- nosy: +

[issue23216] IDLE grep/find/replace source code needs docstrings

Change by miss-islington : -- pull_requests: +12336 stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list m

[issue23216] IDLE grep/find/replace source code needs docstrings

Cheryl Sabella added the comment: New changeset 0bb5e75cf8bc9b197ffb91cba6f30543ed502708 by Cheryl Sabella in branch 'master': bpo-23216: IDLE: Add docstrings to search modules (GH-12141) https://github.com/python/cpython/commit/0bb5e75cf8bc9b197ffb91cba6f30543ed502708 -- _

[issue23216] IDLE grep/find/replace source code needs docstrings

Terry J. Reedy added the comment: There are several minor things to fix after this (and multiple issues), including the difference between find in search and replace. It seems to me that is should be the same. We will definitely need to test changes on both Windows and Linux. The font col

[issue23216] IDLE grep/find/replace source code needs docstrings

Cheryl Sabella added the comment: I've created the first PR for docstrings on replace. Saimadhav Heblikar had added some docstrings under issue 21676, so I expanded on what he did. I referred to Al's diff, but didn't use it much since there were already some docstrings. For readability, I

[issue23216] IDLE grep/find/replace source code needs docstrings

Change by Cheryl Sabella : -- pull_requests: +12142 stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list m

[issue23216] IDLE grep/find/replace source code needs docstrings

Change by Cheryl Sabella : -- assignee: -> cheryl.sabella ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue23216] IDLE grep/find/replace source code needs docstrings

Terry J. Reedy added the comment: Current module names, docstring status, and note. grep - present, some args not doc'ed replace - spotty; do not repetively doc 'event=None' args search - ditto searchbase - present searchengine - present; do not separately docstring 'isxyz' wrappers Closing "

[issue23216] IDLE grep/find/replace source code needs docstrings

Changes by Berker Peksag : -- nosy: +terry.reedy stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue23216] IDLE grep/find/replace source code needs docstrings

Changes by Al Sweigart : -- versions: +Python 2.7, Python 3.4 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue23216] IDLE grep/find/replace source code needs docstrings

New submission from Al Sweigart: The following IDLE files need docstrings for their methods: GrepDialog.py SearchEngine.py SearchDialogBase.py SearchDialog.py ReplaceDialog.py -- components: IDLE files: idle_docstrings.diff keywords: patch messages: 233807 nosy: Al.Sweigart priority: no