[llvm] 428660c - [GitHub] Don't indent comment that revision has passed the formatting check

David Spickett via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 02:55:12 PST 2023


Author: David Spickett
Date: 2023-12-18T10:54:59Z
New Revision: 428660cfb986dd0a59cd2a16972c5f7109080522

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

LOG: [GitHub] Don't indent comment that revision has passed the formatting check

Due to the way the f string was written, the text ended up with 4 spaces
at the start. 4 space indent in Markdown means plain text, which is not
what we intend here.

Added: 
    

Modified: 
    llvm/utils/git/code-format-helper.py

Removed: 
    


################################################################################
diff  --git a/llvm/utils/git/code-format-helper.py b/llvm/utils/git/code-format-helper.py
index 697a27ab82a750..849ae996f7339f 100755
--- a/llvm/utils/git/code-format-helper.py
+++ b/llvm/utils/git/code-format-helper.py
@@ -131,8 +131,8 @@ def run(self, changed_files: List[str], args: FormatArgs) -> bool:
         if 
diff  is None:
             if should_update_gh:
                 comment_text = f"""
-    :white_check_mark: With the latest revision this PR passed the {self.friendly_name}.
-    """
+:white_check_mark: With the latest revision this PR passed the {self.friendly_name}.
+"""
                 self.update_pr(comment_text, args, create_new=False)
             return True
         elif len(
diff ) > 0:


        


More information about the llvm-commits mailing list