Run pep8 everywhere

Since the Zuul v2 files are removed, we can lint pep8 with less changes
everywhere. Enable pep8 linting everywhere and fix remaining problems.

Depends-On: Id67d0149b645bd23e460cef65719541f76f0967b
Change-Id: I0ac0709b05912d00d02a51d732b1cf12eaf888a5
This commit is contained in:
Andreas Jaeger 2017-10-21 18:17:28 +02:00 committed by Andreas Jaeger
parent 4adbecad99
commit 996a0f5b9c
4 changed files with 3 additions and 7 deletions

View File

@ -14,8 +14,6 @@
# License for the specific language governing permissions and limitations
# under the License.
import collections
import ruamel.yaml

View File

@ -62,7 +62,7 @@ while not done:
changes = json.loads(r.text[4:])
for change in changes:
if (not change.get('labels') or
not change.get('labels').get('Verified')):
not change.get('labels').get('Verified')):
continue
for key, value in change['labels']['Verified'].items():
if key == 'value':

View File

@ -31,7 +31,7 @@ def main():
for project in data:
if ('upstream' in project and
'track-upstream' not in project.get('options', [])):
'track-upstream' not in project.get('options', [])):
del project['upstream']
with open('gerrit/projects.yaml', 'w') as out:

View File

@ -10,9 +10,7 @@ passenv = http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY
[testenv:linters]
commands =
{toxinidir}/tools/run-bashate.sh
# TODO(jaegerandi): Remove parameter to test everywhere once
# Zuul v2 files have been removed
flake8 jenkins/scripts roles playbooks nodepool
flake8
[testenv:gerrit]
deps = PyYAML