pep8 returns false N302 for missing import
Bug #1133103 reported by
Joe Gordon
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Joe Gordon |
Bug Description
The pep8 for patch set 6 of https:/
pep8 reported
'./nova/
because https:/
had in invalid import.
The current logic compares the last word of the module name along with the last word of the exception msg, and if they are not the same a module is missing. This check should also make sure the words "cannot import" aren't in the exception msg.
Changed in nova: | |
milestone: | none → grizzly-rc1 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | grizzly-rc1 → 2013.1 |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/22947
Review: https:/