Fix E275: missing whitespace around keywords
This commit is contained in:
parent
b3ea77e966
commit
b40d8c0053
@ -130,8 +130,8 @@ class CheckSource(ReviewBot.ReviewBot):
|
||||
self.logger.info('checking if target package exists and has devel project')
|
||||
devel_project, devel_package = devel_project_get(self.apiurl, target_project, target_package)
|
||||
if devel_project:
|
||||
if (source_project != devel_project or source_package != devel_package) and \
|
||||
not(source_project == target_project and source_package == target_package):
|
||||
if ((source_project != devel_project or source_package != devel_package) and
|
||||
not (source_project == target_project and source_package == target_package)):
|
||||
# Not from proper devel project/package and not self-submission.
|
||||
self.review_messages['declined'] = 'Expected submission from devel package %s/%s' % (
|
||||
devel_project, devel_package)
|
||||
|
@ -145,7 +145,7 @@ def prompt_interactive(changes, project, package):
|
||||
|
||||
|
||||
def issue_found(package, label, db):
|
||||
return not(package not in db or db[package] is None or label not in db[package])
|
||||
return not (package not in db or db[package] is None or label not in db[package])
|
||||
|
||||
|
||||
def issue_trackers(apiurl):
|
||||
|
@ -12,7 +12,7 @@ MINIMALS = {
|
||||
x.rstrip()
|
||||
for x in requests.get(
|
||||
'https://gitlab.suse.de/qa-maintenance/metadata/raw/master/packages-to-be-tested-on-minimal-systems').iter_lines()
|
||||
if len(x) > 0 and not(x.startswith(b"#") or x.startswith(b' '))}
|
||||
if len(x) > 0 and not (x.startswith(b"#") or x.startswith(b' '))}
|
||||
|
||||
|
||||
class SUSEUpdate(Update):
|
||||
|
@ -65,8 +65,9 @@ def _full_project_name(self, project):
|
||||
|
||||
|
||||
def lock_needed(cmd, opts):
|
||||
return not(
|
||||
cmd in ('check', 'check_duplicate_binaries', 'check_local_links', 'frozenage', 'rebuild', 'unlock', 'setprio') or
|
||||
return not (
|
||||
cmd in ('check', 'check_duplicate_binaries', 'check_local_links',
|
||||
'frozenage', 'rebuild', 'unlock', 'setprio') or
|
||||
(cmd == 'list' and not opts.supersede)
|
||||
)
|
||||
|
||||
|
@ -97,7 +97,7 @@ class CommentAPI(object):
|
||||
if info_match:
|
||||
match = True
|
||||
for key, value in info_match.items():
|
||||
if not(value is None or (key in info and info[key] == value)):
|
||||
if not (value is None or (key in info and info[key] == value)):
|
||||
match = False
|
||||
break
|
||||
if not match:
|
||||
|
@ -450,7 +450,7 @@ class StagingAPI(object):
|
||||
is_targeted = (target_package in target_requests or
|
||||
str(request_id) in target_requests)
|
||||
if action.get('type') in ['submit', 'delete'] and (
|
||||
not(target_requests) or is_targeted):
|
||||
not (target_requests) or is_targeted):
|
||||
stage_info = self.packages_staged.get(target_package)
|
||||
|
||||
# Ensure a request for same package is already staged.
|
||||
|
Loading…
x
Reference in New Issue
Block a user