[clang] [llvm] [WIP][GitHub][CI] Add running of dump_ast_matchers.py to premerge workflow (PR #165472)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 28 13:40:55 PDT 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {darker}-->
:warning: Python code formatter, darker found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
darker --check --diff -r origin/main...HEAD llvm/utils/git/code-format-helper.py
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from darker here.
</summary>
``````````diff
--- code-format-helper.py 2025-10-28 20:39:06.000000 +0000
+++ code-format-helper.py 2025-10-28 20:40:24.212537 +0000
@@ -497,12 +497,16 @@
if args.verbose:
print(f"=== Debug: DumpASTMatchersHelper ===")
print(f"Current working directory: {os.getcwd()}")
print(f"Script directory: {self.script_dir}")
print(f"Script name: {self.script_name}")
- print(f"Script full path: {os.path.join(self.script_dir, self.script_name)}")
- print(f"Script exists: {os.path.exists(os.path.join(self.script_dir, self.script_name))}")
+ print(
+ f"Script full path: {os.path.join(self.script_dir, self.script_name)}"
+ )
+ print(
+ f"Script exists: {os.path.exists(os.path.join(self.script_dir, self.script_name))}"
+ )
print(f"Output HTML path: {self.output_html}")
print(f"Output HTML exists: {os.path.exists(self.output_html)}")
# Verify the input file exists
matchers_header = "clang/include/clang/ASTMatchers/ASTMatchers.h"
@@ -554,18 +558,22 @@
)
if args.verbose:
print(f"Git diff exit code: {proc.returncode}")
if proc.returncode != 0:
- print(f"Output HTML was modified (diff length: {len(proc.stdout)} chars)")
+ print(
+ f"Output HTML was modified (diff length: {len(proc.stdout)} chars)"
+ )
else:
print(f"Output HTML was NOT modified - check passed!")
# 'LibASTMatchersReference.html' was modified - count as failure
if proc.returncode != 0:
if args.verbose:
- print(f"error: {self.name} detected that {self.output_html} was modified")
+ print(
+ f"error: {self.name} detected that {self.output_html} was modified"
+ )
print(f"This means the developer needs to run the script locally")
print(f"Diff preview (first 500 chars):\n{proc.stdout[:500]}")
return proc.stdout
else:
return None
``````````
</details>
https://github.com/llvm/llvm-project/pull/165472
More information about the cfe-commits
mailing list