➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2017-11-28 15:07 by vstinner, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 4614 | closed | vstinner, 2017-11-28 15:19 | |
| Messages (4) | |||
|---|---|---|---|
| msg307126 - (view) | Author: STINNER Victor (vstinner) *  |
Date: 2017-11-28 15:07 | |
Attached PR removes unused imports. |
|||
| msg307141 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2017-11-28 16:18 | |
I suggest to split your patch on several PR for different domains, and let the domain experts review changes separately. IDLE changes look mostly harmless to me. But changes in distutils should be made with a large care. |
|||
| msg307205 - (view) | Author: Terry J. Reedy (terry.reedy) * |
Date: 2017-11-29 03:18 | |
Idlelib deletions done. pathbrowser, #32100, PR-4626 configdialog, #30781, PR-4627 |
|||
| msg318106 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2018-05-29 21:52 | |
> I suggest to split your patch on several PR for different domains, and let the domain experts review changes separately. Right. I should recreate multiple PRs with fewer changes per PR. I close this issue. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:54 | admin | set | github: 76337 |
| 2018-05-29 21:52:08 | vstinner | set | status: open -> closed resolution: out of date messages: + msg318106 stage: patch review -> resolved |
| 2017-11-29 03:18:12 | terry.reedy | set | nosy:
+ terry.reedy messages: + msg307205 |
| 2017-11-28 16:18:49 | serhiy.storchaka | set | nosy:
+ serhiy.storchaka messages: + msg307141 |
| 2017-11-28 15:19:31 | vstinner | set | keywords:
+ patch stage: patch review pull_requests: + pull_request4529 |
| 2017-11-28 15:07:16 | vstinner | create | |