pep8 tests has a lot of disabled rules
Bug #1333290 reported by
Valeriy Ponomaryov
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Shared File Systems Service (Manila) |
Fix Released
|
Medium
|
Andreas Jaeger |
Bug Description
pep8/flake8 tests in manila has a lot of skipped rules.
We need to fix problem places and enable rules, without it we have newly added code, that does not correspond to these rules.
Changed in manila: | |
importance: | Undecided → Medium |
tags: | added: manila manilaclient |
Changed in manila: | |
status: | New → In Progress |
tags: | removed: manilaclient |
description: | updated |
Changed in manila: | |
assignee: | nobody → Andreas Jaeger (jaegerandi) |
Changed in manila: | |
milestone: | none → juno-3 |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/114486 /git.openstack. org/cgit/ stackforge/ manila/ commit/ ?id=09b1917fe89 e26e318ed2f341d a6c2c3b3fb4191
Committed: https:/
Submitter: Jenkins
Branch: master
commit 09b1917fe89e26e 318ed2f341da6c2 c3b3fb4191
Author: Andreas Jaeger <email address hidden>
Date: Fri Aug 15 11:40:56 2014 +0200
Fix pep8 F811 and F812
Fixes: scheduler/ simple. py:74:45: F812 list comprehension redefines api/v1/ share_networks. py:147: 41: F811 redefinition of unused
manila/
'service' from line 59
manila/
'network' from line 29
Do not blacklist these tests anymore.
Partial-Bug: #1333290 af0708331331bc4 35942c3763b
Change-Id: Ibf3c170ada17a7