Use line break before binary operators in Python code

PEP 8 recommends line break before binary operators.

Author: per1234

reportsizedeltas.py

@@ -121,8 +121,8 @@ def report_exists(self, pr_number, pr_head_sha):
         page_number = 1
         page_count = 1
         while page_number <= page_count:
-            api_data = self.api_request(request="repos/" + self.repository_name + "/issues/" + str(pr_number) +
-                                                "/comments",
+            api_data = self.api_request(request="repos/" + self.repository_name + "/issues/" + str(pr_number)
+                                                + "/comments",
                                         page_number=page_number)
 
             comments_data = api_data["json_data"]
@@ -150,8 +150,8 @@ def get_artifact_download_url_for_sha(self, pr_user_login, pr_head_ref, pr_head_
         page_count = 1
         while page_number <= page_count:
             api_data = self.api_request(request="repos/" + self.repository_name + "/actions/runs",
-                                        request_parameters="actor=" + pr_user_login + "&branch=" + pr_head_ref +
-                                                           "&event=pull_request&status=completed",
+                                        request_parameters="actor=" + pr_user_login + "&branch=" + pr_head_ref
+                                                           + "&event=pull_request&status=completed",
                                         page_number=page_number)
             runs_data = api_data["json_data"]
 
@@ -179,8 +179,8 @@ def get_artifact_download_url_for_run(self, run_id):
         page_number = 1
         page_count = 1
         while page_number <= page_count:
-            api_data = self.api_request(request="repos/" + self.repository_name + "/actions/runs/" +
-                                                str(run_id) + "/artifacts",
+            api_data = self.api_request(request="repos/" + self.repository_name + "/actions/runs/"
+                                                + str(run_id) + "/artifacts",
                                         page_number=page_number)
             artifacts_data = api_data["json_data"]
 
@@ -230,19 +230,19 @@ def generate_report(self, artifact_folder_object, pr_head_sha, pr_number):
         artifact_folder_object -- object containing the data about the temporary folder that stores the markdown files
         """
         with artifact_folder_object as artifact_folder:
-            report_markdown = (self.report_key_beginning + pr_head_sha + "]" +
-                               "(https://github.com/" + self.repository_name + "/pull/" + str(pr_number) +
-                               "/commits/" + pr_head_sha + ")**\n\n")
+            report_markdown = (self.report_key_beginning + pr_head_sha + "]"
+                               + "(https://github.com/" + self.repository_name + "/pull/" + str(pr_number)
+                               + "/commits/" + pr_head_sha + ")**\n\n")
             report_markdown = report_markdown + "FQBN | Flash Usage | RAM For Global Variables\n---|---|---"
             reports_data = []
             for report_filename in sorted(os.listdir(path=artifact_folder)):
                 with open(file=artifact_folder + "/" + report_filename) as report_file:
                     report_data = json.load(report_file)
                     reports_data = reports_data + [report_data]
-                    report_markdown = (report_markdown + "\n" +
-                                       report_data["fqbn"] +
-                                       generate_value_cell(report_data["flash_delta"]) +
-                                       generate_value_cell(report_data["ram_delta"]))
+                    report_markdown = (report_markdown + "\n"
+                                       + report_data["fqbn"]
+                                       + generate_value_cell(report_data["flash_delta"])
+                                       + generate_value_cell(report_data["ram_delta"]))
 
         logger.debug("Report:\n" + report_markdown)
         return {"markdown": report_markdown, "data": reports_data}
@@ -257,11 +257,11 @@ def comment_report(self, pr_number, report_markdown):
         report_data = {"body": report_markdown}
         report_data = json.dumps(obj=report_data)
         report_data = report_data.encode(encoding="utf-8")
-        url = ("https://api.github.com/repos/" +
-               self.repository_name +
-               "/issues/" +
-               str(pr_number) +
-               "/comments")
+        url = ("https://api.github.com/repos/"
+               + self.repository_name
+               + "/issues/"
+               + str(pr_number)
+               + "/comments")
 
         self.http_request(url=url, data=report_data)
 
@@ -278,8 +278,8 @@ def api_request(self, request, request_parameters="", page_number=1):
         page_number -- Some responses will be paginated. This argument specifies which page should be returned.
                        (default value: 1)
         """
-        return self.get_json_response(url="https://api.github.com/" + request + "?" + request_parameters + "&page=" +
-                                          str(page_number) + "&per_page=100")
+        return self.get_json_response(url="https://api.github.com/" + request + "?" + request_parameters + "&page="
+                                          + str(page_number) + "&per_page=100")
 
     def get_json_response(self, url):
         """Load the specified URL and return a dictionary:

tests/test_reportsizedeltas.py

@@ -99,8 +99,8 @@ def test_report_exists(self):
 
         self.assertTrue(report_size_deltas.report_exists(pr_number=pr_number, pr_head_sha=pr_head_sha))
 
-        report_size_deltas.api_request.assert_called_once_with(request="repos/" + repository_name + "/issues/" +
-                                                                       str(pr_number) + "/comments",
+        report_size_deltas.api_request.assert_called_once_with(request="repos/" + repository_name + "/issues/"
+                                                                       + str(pr_number) + "/comments",
                                                                page_number=1)
 
         self.assertFalse(report_size_deltas.report_exists(pr_number=pr_number, pr_head_sha="asdf"))
@@ -129,8 +129,8 @@ def test_get_artifact_download_url_for_sha(self):
 
         # Test pagination
         request = "repos/" + repository_name + "/actions/runs"
-        request_parameters = ("actor=" + pr_user_login + "&branch=" + pr_head_ref +
-                              "&event=pull_request&status=completed")
+        request_parameters = ("actor=" + pr_user_login + "&branch=" + pr_head_ref
+                              + "&event=pull_request&status=completed")
         calls = [unittest.mock.call(request=request, request_parameters=request_parameters, page_number=1),
                  unittest.mock.call(request=request, request_parameters=request_parameters, page_number=2),
                  unittest.mock.call(request=request, request_parameters=request_parameters, page_number=3)]
@@ -170,8 +170,8 @@ def test_get_artifact_download_url_for_run(self):
         self.assertEqual(archive_download_url, report_size_deltas.get_artifact_download_url_for_run(run_id=run_id))
 
         report_size_deltas.api_request.assert_called_once_with(
-            request="repos/" + repository_name + "/actions/runs/" + str(run_id) +
-                    "/artifacts",
+            request="repos/" + repository_name + "/actions/runs/" + str(run_id)
+                    + "/artifacts",
             page_number=1)
 
         json_data = {"artifacts": [{"name": "foo123", "archive_download_url": "test_artifact_url"},
@@ -204,9 +204,9 @@ def test_generate_report(self):
         report = report_size_deltas.generate_report(artifact_folder_object=artifact_folder_object,
                                                     pr_head_sha=pr_head_sha, pr_number=pr_number)
         report_markdown = (
-                report_size_deltas.report_key_beginning + pr_head_sha +
-                "](https://github.com/" + repository_name + "/pull/" + str(pr_number) + "/commits/" + pr_head_sha +
-                ")**\n\n"
+                report_size_deltas.report_key_beginning + pr_head_sha
+                + "](https://github.com/" + repository_name + "/pull/" + str(pr_number) + "/commits/" + pr_head_sha
+                + ")**\n\n"
                 "FQBN | Flash Usage | RAM For Global Variables\n"
                 "---|---|---\n"
                 "adafruit:samd:adafruit_feather_m0 | 0 | N/A\n"
@@ -266,8 +266,8 @@ def test_comment_report(self):
         report_data = report_data.encode(encoding="utf-8")
 
         report_size_deltas.http_request.assert_called_once_with(
-            url="https://api.github.com/repos/" + repository_name + "/issues/" +
-                str(pr_number) + "/comments",
+            url="https://api.github.com/repos/" + repository_name + "/issues/"
+                + str(pr_number) + "/comments",
             data=report_data)
 
     # @unittest.skip("")
@@ -287,8 +287,8 @@ def test_api_request(self):
                                                                        request_parameters=request_parameters,
                                                                        page_number=page_number))
         report_size_deltas.get_json_response.assert_called_once_with(
-            url="https://api.github.com/" + request + "?" + request_parameters +
-                "&page=" + str(page_number) + "&per_page=100")
+            url="https://api.github.com/" + request + "?" + request_parameters
+                + "&page=" + str(page_number) + "&per_page=100")
 
     # @unittest.skip("")
     def test_get_json_response(self):