should not exclude plugins from flake8 tests
Bug #1197858 reported by
John Garbutt
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
High
|
John Garbutt |
Bug Description
We have pep8 errors sneeking into all the plugins due to the plugins being excluded from pep8 checks.
Unfortunatly, this includes both:
/plugins/xenserver
and
/nova/api/
We want to exclude the top one, but not the bottom ones.
Changed in nova: | |
importance: | Undecided → High |
assignee: | nobody → John Garbutt (johngarbutt) |
status: | New → Triaged |
Changed in nova: | |
status: | Triaged → In Progress |
description: | updated |
summary: |
- should not exclude plugins from pep8 + should not exclude plugins from flake8 tests |
Changed in nova: | |
milestone: | none → havana-2 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | havana-2 → 2013.2 |
To post a comment you must log in.
We can't check the the xenserver plugins because they are run in python 2.4, and our flake8 settings enforce the python 2.4 incompatible:
N308 Python 3.x incompatible 'except x,y:' construct