PEP8 version differs between test_requires and tox.ini
Bug #1037303 reported by
Maru Newby
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Identity (keystone) |
Fix Released
|
Low
|
Dolph Mathews | ||
Bug Description
tox.ini specifies pep8==1.1, but test_requires specifies 1.3.3. This means that a tox pep8 run may (and currently does) return different results than a run_tests.sh pep8 run.
Presumably the version in tox.ini should be updated to match test_requires?
Revision history for this message
| Joseph Heck (heckj) wrote | #1 |
| Changed in keystone: | |
| importance: | Undecided → Low |
| status: | New → Triaged |
Revision history for this message
| Maru Newby (maru) wrote | #2 |
| Changed in keystone: | |
| assignee: | nobody → Dolph Mathews (dolph) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to keystone (master) | #3 |
| Changed in keystone: | |
| status: | Triaged → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to keystone (master) | #4 |
| Changed in keystone: | |
| status: | In Progress → Fix Committed |
| Changed in keystone: | |
| milestone: | none → folsom-rc1 |
| Changed in keystone: | |
| status: | Fix Committed → Fix Released |
| Changed in keystone: | |
| milestone: | folsom-rc1 → 2012.2 |
To post a comment you must log in.
Until we do all the pep8 1.3 cleanup (mostly indent related work), the tox.ini shouldn't be updated. It's not adding any functionality, so we've deferred this work from earlier.