➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2017-06-26 21:57 byterry.reedy, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 2440 | merged | cheryl.sabella,2017-06-27 12:58 | |
| PR 2576 | merged | terry.reedy,2017-07-05 02:48 | |
| PR 2577 | merged | terry.reedy,2017-07-05 03:02 | |
| PR 2579 | merged | terry.reedy,2017-07-05 03:05 | |
| Messages (6) | |||
|---|---|---|---|
| msg296955 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-06-26 21:57 | |
Spinoff from#30728. Docstrings are a form of comment. Both leave behavior alone and do not require testing, just human review. Add the docstrings from the original version ofPR 2307, and any others. Review and fix comments in light of the comments on that PR. | |||
| msg297030 -(view) | Author: Cheryl Sabella (cheryl.sabella)* | Date: 2017-06-27 13:01 | |
I've made a PR for the docstrings and comments. I didn't change any of the comments separating the sections in the create* functions as I wasn't sure how you wanted those changed. When reading the code, I found the comments helpful in that they separated each section, but whitespace might be just as helpful, visually.Thanks! | |||
| msg297059 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-06-27 17:22 | |
Louie, perhaps you would like to review the PR, especially the comment revisions responding to your comments. | |||
| msg297693 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-07-05 01:31 | |
New changeset7eb5883ac59833bf63f0e1f7fb95671a1ac1ee08 by terryjreedy (csabella) in branch 'master':bpo-30777: IDLE: configdialog - add docstrings and improve comments (#2440)https://github.com/python/cpython/commit/7eb5883ac59833bf63f0e1f7fb95671a1ac1ee08 | |||
| msg297699 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-07-05 04:54 | |
New changesete5bb112ba2048cb3ea19481a7b38fe1433c5a7ef by terryjreedy in branch '3.6':[3.6]bpo-30777: IDLE: configdialog - add docstrings and improve comments (GH-2440) (#2577)https://github.com/python/cpython/commit/e5bb112ba2048cb3ea19481a7b38fe1433c5a7ef | |||
| msg297778 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2017-07-05 20:21 | |
Unlinked bogus revert message. configdialog still has new docstrings. | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:48 | admin | set | github: 74960 |
| 2017-07-05 20:21:52 | terry.reedy | set | versions: + Python 3.7 |
| 2017-07-05 20:21:04 | terry.reedy | set | messages: +msg297778 |
| 2017-07-05 20:20:34 | terry.reedy | set | messages: -msg297741 |
| 2017-07-05 13:44:55 | vstinner | set | nosy: +vstinner messages: +msg297741 |
| 2017-07-05 04:56:34 | terry.reedy | set | status: open -> closed resolution: fixed stage: needs patch -> resolved |
| 2017-07-05 04:54:58 | terry.reedy | set | messages: +msg297699 |
| 2017-07-05 03:05:10 | terry.reedy | set | pull_requests: +pull_request2649 |
| 2017-07-05 03:02:11 | terry.reedy | set | pull_requests: +pull_request2647 |
| 2017-07-05 02:48:06 | terry.reedy | set | pull_requests: +pull_request2646 |
| 2017-07-05 01:31:03 | terry.reedy | set | messages: +msg297693 |
| 2017-06-27 17:22:18 | terry.reedy | set | nosy:terry.reedy,louielu,cheryl.sabella messages: +msg297059 |
| 2017-06-27 13:01:52 | cheryl.sabella | set | messages: +msg297030 |
| 2017-06-27 12:58:51 | cheryl.sabella | set | pull_requests: +pull_request2489 |
| 2017-06-27 00:46:41 | terry.reedy | link | issue30728 dependencies |
| 2017-06-26 22:41:15 | terry.reedy | set | assignee:terry.reedy components: + IDLE versions: + Python 3.6 |
| 2017-06-26 21:57:01 | terry.reedy | create | |