[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread STINNER Victor
New submission from STINNER Victor : Attached PR removes unused imports. -- components: Library (Lib) messages: 307126 nosy: vstinner priority: normal severity: normal status: open title: Fix flake8 warning F401: ... imported but unused versions: Python 3.7 _

[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- keywords: +patch pull_requests: +4529 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-li

[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: 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. -- nosy: +serhiy.storchaka

[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: Idlelib deletions done. pathbrowser, #32100, PR-4626 configdialog, #30781, PR-4627 -- nosy: +terry.reedy ___ Python tracker ___ __

[issue32156] Fix flake8 warning F401: ... imported but unused

2018-05-29 Thread STINNER Victor
STINNER Victor added the comment: > 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. -- resolution: -> out of date stage: p