N302 is broken in hacking.py
Bug #1130411 reported by
Joe Gordon
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Vish Ishaya |
Bug Description
http://
Says "import only modules.'from oslo.config import cfg' does not import a module"
but python shows cfg is a module
>>> from oslo.config import cfg
>>> isinstance(cfg, types.ModuleType)
True
Changed in nova: | |
assignee: | nobody → Vish Ishaya (vishvananda) |
status: | New → In Progress |
Changed in nova: | |
milestone: | none → grizzly-3 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | grizzly-3 → 2013.1 |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/22385 github. com/openstack/ nova/commit/ 3b649422a61dee8 22d813c6fbf9afb 46569587b2
Committed: http://
Submitter: Jenkins
Branch: master
commit 3b649422a61dee8 22d813c6fbf9afb 46569587b2
Author: Vishvananda Ishaya <email address hidden>
Date: Tue Feb 19 16:59:53 2013 -0800
Fix hacking test to handle namespace packages.
This removes our workaround for namespace packages and improves
detection of missing imports due to non-existent dependencies.
Fixes bug 1130411
Change-Id: I89ee8249580e39 5f786289792cdf7 4219d041f2f