Issue 32156: Fix flake8 warning F401: ... imported but unused (original) (raw)

Issue32156

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) * (Python committer) Date: 2017-11-28 15:07
Attached PR removes unused imports.
msg307141 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) 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) * (Python committer) Date: 2017-11-29 03:18
Idlelib deletions done. pathbrowser, #32100, PR-4626 configdialog, #30781, PR-4627
msg318106 - (view) Author: STINNER Victor (vstinner) * (Python committer) 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 -> closedresolution: out of datemessages: + stage: patch review -> resolved
2017-11-29 03🔞12 terry.reedy set nosy: + terry.reedymessages: +
2017-11-28 16🔞49 serhiy.storchaka set nosy: + serhiy.storchakamessages: +
2017-11-28 15:19:31 vstinner set keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest4529>
2017-11-28 15:07:16 vstinner create