summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJames E. Blair <jim@acmegating.com>2022-08-02 08:02:30 -0700
committerJames E. Blair <jim@acmegating.com>2022-08-02 08:02:30 -0700
commitfbb97bd6b6d039a81279fd01a66f9c7c6698a855 (patch)
tree411cf376d43ae72808349460c8822f30cc23a73a
parentde1e2a325e76b573544f5c1f4bf92e73c675e645 (diff)
downloadzuul-fbb97bd6b6d039a81279fd01a66f9c7c6698a855.tar.gz
Add whitespace around keywords
This fixes pep8 E275 which wants whitespace after assert and del. Change-Id: I1f8659f462aa91c3fdf8f7eb8b939b67c0ce9f55
-rw-r--r--tests/unit/test_github_driver.py2
-rw-r--r--zuul/driver/gerrit/gerritconnection.py2
-rw-r--r--zuul/executor/server.py2
3 files changed, 3 insertions, 3 deletions
diff --git a/tests/unit/test_github_driver.py b/tests/unit/test_github_driver.py
index f9f3371fa..49dae0ccb 100644
--- a/tests/unit/test_github_driver.py
+++ b/tests/unit/test_github_driver.py
@@ -1384,7 +1384,7 @@ class TestGithubDriver(ZuulTestCase):
# now check if the merge was done via rebase
merges = [report for report in self.fake_github.github_data.reports
if report[2] == 'merge']
- assert(len(merges) == 1 and merges[0][3] == 'squash')
+ assert (len(merges) == 1 and merges[0][3] == 'squash')
@simple_layout('layouts/basic-github.yaml', driver='github')
def test_invalid_event(self):
diff --git a/zuul/driver/gerrit/gerritconnection.py b/zuul/driver/gerrit/gerritconnection.py
index 6aea4388b..1ec334915 100644
--- a/zuul/driver/gerrit/gerritconnection.py
+++ b/zuul/driver/gerrit/gerritconnection.py
@@ -1470,7 +1470,7 @@ class GerritConnection(ZKChangeCacheMixin, ZKBranchCacheMixin, BaseConnection):
# for large projects like nova
alldata = []
chunk, more_changes = _query_chunk(query, event)
- while(chunk):
+ while chunk:
alldata.extend(chunk)
if more_changes is None:
# continue sortKey based (before Gerrit 2.9)
diff --git a/zuul/executor/server.py b/zuul/executor/server.py
index 2165a797b..eac7fa7e5 100644
--- a/zuul/executor/server.py
+++ b/zuul/executor/server.py
@@ -3735,7 +3735,7 @@ class ExecutorServer(BaseMergeServer):
sensor.reportStats(self.statsd, base_key)
def finishJob(self, unique):
- del(self.job_workers[unique])
+ del self.job_workers[unique]
self.log.debug(
"Finishing Job: %s, queue(%d): %s",
unique,