[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thanks Berker. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed type: -> enhancement ___ Python tracker ___ __

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Roundup Robot
Roundup Robot added the comment: New changeset fa44d1bc9b68 by Serhiy Storchaka in branch 'default': Issue #23277: Remove unused imports in tests. https://hg.python.org/cpython/rev/fa44d1bc9b68 -- ___ Python tracker

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Berker Peksag
Berker Peksag added the comment: LGTM -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.or

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Here is the patch that contains the rest of changes, after resolving conflicts and fixing some errors. I have reviewed and tested these changes and they LGTM. I think it is worth to push them. -- resolution: fixed -> status: closed -> open Added fil

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9d7f2615f7b3 by Serhiy Storchaka in branch 'default': Issue #23277: Remove more unused sys and os imports. https://hg.python.org/cpython/rev/9d7f2615f7b3 New changeset abf3f0dcf2fd by Serhiy Storchaka in branch 'default': Issue #23277: Remove unused

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-24 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Note that setUpModule is correctly imported in Lib/tkinter/test/test_tkinter/test_widgets.py. -- nosy: +serhiy.storchaka ___ Python tracker _

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-23 Thread Berker Peksag
Berker Peksag added the comment: Thanks for the patch, Jon. I only removed sys and os since they were the most unused modules. -- resolution: -> fixed status: open -> closed versions: +Python 3.6 -Python 3.4, Python 3.5 ___ Python tracker

[issue23277] Cleanup unused and duplicate imports in tests

2016-04-23 Thread Roundup Robot
Roundup Robot added the comment: New changeset 6958bbf7f0ec by Berker Peksag in branch 'default': Issue #23277: Remove unused sys and os imports https://hg.python.org/cpython/rev/6958bbf7f0ec -- nosy: +python-dev ___ Python tracker

[issue23277] Cleanup unused and duplicate imports in tests

2015-01-19 Thread Berker Peksag
Berker Peksag added the comment: +1 for cleanup. -- nosy: +berker.peksag stage: -> patch review versions: +Python 3.4, Python 3.5 -Python 3.6 ___ Python tracker ___

[issue23277] Cleanup unused and duplicate imports in tests

2015-01-19 Thread Jon Dufresne
New submission from Jon Dufresne: Ran variations of the command: $ find . -wholename '*/test/*.py' | xargs flake8 --select=F401,F811 To look for unused or duplicate imports. The attached patch removes them. -- components: Tests files: cleanup-unused-imports.patch keywords: patch messag