Tool for checking unused imports needed
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
oslo-incubator |
Fix Released
|
Medium
|
Zhongyue Luo |
Bug Description
Currently there are some unused imports in the upstream codebase.
************* Module tests.utils
W: 20,0: Unused import subprocess
************* Module tests.unit.
W: 18,0: Unused import cfg
************* Module tests.unit.
W: 23,0: Unused import rabbit_notifier
************* Module tests.unit.
...
Instead of conducting janitorial cleanups every now and then we should have pylint check for unused imports for each patch.
Therefore adding a python script that executes pylint and a pylint section to tox.ini should prevent unused imports perminantly.
The reason writing a script is for flexible configuration of pylint which would be useful for adding exceptions in the future.
description: | updated |
Changed in oslo: | |
importance: | Undecided → Medium |
Changed in oslo: | |
milestone: | none → havana-1 |
status: | Fix Committed → Fix Released |
Changed in oslo: | |
milestone: | havana-1 → 2013.2 |
Fix proposed to branch: master /review. openstack. org/24882
Review: https:/