run_tests should run pep8 by default
Bug #911438 reported by
Ed Leafe
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Core Infrastructure |
Fix Released
|
Undecided
|
Unassigned | ||
OpenStack Identity (keystone) |
Fix Released
|
Undecided
|
Ed Leafe |
Bug Description
Just got burned because my last trivial change introduced a pep8 violation. In nova, run_tests.sh runs the pep8 check if all tests pass; we should do this in keystone, too.
Changed in keystone: | |
status: | New → In Progress |
assignee: | nobody → Ed Leafe (ed-leafe) |
Changed in keystone: | |
status: | Fix Committed → Fix Released |
Changed in keystone: | |
milestone: | none → essex-3 |
Changed in keystone: | |
milestone: | essex-3 → 2012.1 |
To post a comment you must log in.
I will add a command-line option to skip the pep8 tests, as Ziad has informed me that gerrit runs pep8 separately.