[llvm] dcbbfb2 - [Utils] Format git-llvm-push

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 15 11:52:18 PDT 2026


Author: Aiden Grossman
Date: 2026-03-15T18:51:26Z
New Revision: dcbbfb26cb2a1aad906af7e46ec1e81cbae89256

URL: https://github.com/llvm/llvm-project/commit/dcbbfb26cb2a1aad906af7e46ec1e81cbae89256
DIFF: https://github.com/llvm/llvm-project/commit/dcbbfb26cb2a1aad906af7e46ec1e81cbae89256.diff

LOG: [Utils] Format git-llvm-push

Use single quotes for string arguments inside f-strings or otherwise the
version of black that we use fails to parse. Also reformat the file
given that hasn't been working for a while (wholesale or incrementally)
to the above issue.

Added: 
    

Modified: 
    llvm/utils/git-llvm-push

Removed: 
    


################################################################################
diff  --git a/llvm/utils/git-llvm-push b/llvm/utils/git-llvm-push
index 2297215eb4657..9cc062f90fbf5 100644
--- a/llvm/utils/git-llvm-push
+++ b/llvm/utils/git-llvm-push
@@ -204,7 +204,9 @@ class GitHubAPI:
         draft: bool,
     ) -> tuple:
         if self.runner.dry_run:
-            self.runner.print(f"[Dry Run] Would create pull request for '{head_branch}'...")
+            self.runner.print(
+                f"[Dry Run] Would create pull request for '{head_branch}'..."
+            )
             return 0
 
         self.runner.print(f"Creating pull request for '{head_branch}'...")
@@ -218,7 +220,7 @@ class GitHubAPI:
         response_data = self._request_and_parse_json(
             "POST", f"/repos/{LLVM_REPO}/pulls", json_payload=data
         )
-        self.runner.print(f"Pull request created: {response_data.get("html_url")}")
+        self.runner.print(f"Pull request created: {response_data.get('html_url')}")
         return (response_data.get("node_id"), response_data.get("number"))
 
     def get_repo_settings(self) -> dict:
@@ -236,13 +238,16 @@ class GitHubAPI:
         labels: List[str],
     ) -> None:
         if self.runner.dry_run:
-            self.runner.print(f"[Dry Run] Would set labels for #{pr_number}: {' '.join(labels)}")
+            self.runner.print(
+                f"[Dry Run] Would set labels for #{pr_number}: {' '.join(labels)}"
+            )
             return None
 
         self.runner.print(f"Setting labels for #{pr_number}: {' '.join(labels)}")
 
         self._request_and_parse_json(
-            "POST", f"/repos/{LLVM_REPO}/issues/{pr_number}/labels",
+            "POST",
+            f"/repos/{LLVM_REPO}/issues/{pr_number}/labels",
             json_payload={"labels": labels},
         )
 
@@ -630,9 +635,9 @@ def main() -> None:
     )
     parser.add_argument(
         "--labels",
-        nargs='*',
+        nargs="*",
         default=[DEFAULT_LABEL],
-        help=f"Set the PR labels (default: {DEFAULT_LABEL})."
+        help=f"Set the PR labels (default: {DEFAULT_LABEL}).",
     )
     merging = parser.add_mutually_exclusive_group()
     merging.add_argument(


        


More information about the llvm-commits mailing list