From 315b954a1a35f9f03ab2258caf416c01525173a5 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 20 2020 13:17:43 +0000 Subject: flake8 and black fixes for the entire project Signed-off-by: Pierre-Yves Chibon --- diff --git a/runner.py b/runner.py index 20c99f6..5c68a87 100644 --- a/runner.py +++ b/runner.py @@ -13,10 +13,13 @@ import uuid import fedora_messaging.api import fedora_messaging.exceptions -import toml -import monitor_gating_single_build import monitor_gating_multi_builds + +import monitor_gating_single_build + +import toml + from utils import MonitoringUtils, blocking_issues, run_command s = sched.scheduler(time.time, time.sleep) @@ -51,7 +54,7 @@ def _clean_up_side_tags(utils): try: print(" Removing side-tag: %s" % utils.side_tag_name) cmd = [conf["fedpkg"], "remove-side-tag", utils.side_tag_name] - output = run_command(cmd) + run_command(cmd) except Exception: pass @@ -74,7 +77,7 @@ def schedule(conf): single_args = conf["workflow_single_gating_args"].split() notify( topic=f"single-build.start", - message={"arguments": single_args, "runid": runid,}, + message={"arguments": single_args, "runid": runid}, ) output = monitor_gating_single_build.main(single_args) output_text = "\n".join(output) @@ -97,7 +100,7 @@ def schedule(conf): monit_utils = MonitoringUtils() notify( topic=f"multi-build.start", - message={"arguments": multi_args, "runid": runid,}, + message={"arguments": multi_args, "runid": runid}, ) try: output = monitor_gating_multi_builds.main(multi_args, utils=monit_utils) @@ -125,7 +128,7 @@ def schedule(conf): print(sys.exc_info()[0]) notify( - topic=f"multi-build.end.error", message={"runid": runid, "exception": err,}, + topic=f"multi-build.end.error", message={"runid": runid, "exception": err}, ) blocking_issues_list = blocking_issues(blocker_tags) diff --git a/utils.py b/utils.py index 265b2e2..411a7a2 100644 --- a/utils.py +++ b/utils.py @@ -172,7 +172,7 @@ class MonitoringUtils: info_log = f"Creating PR from forks/{username}/{namespace}/{name}" self.print_user(info_log) url = "/".join( - [base_url.rstrip("/"), "api/0", namespace, name, "pull-request/new",] + [base_url.rstrip("/"), "api/0", namespace, name, "pull-request/new"] ) data = { "branch_to": branch, @@ -258,9 +258,7 @@ class MonitoringUtils: "koji", ] if koji_url: - command.extend( - ["-s", koji_url,] - ) + command.extend(["-s", koji_url]) command.extend(["call", "listTags", nevr]) success = False