[libcxx-commits] [libcxx] [libc++] Run clang-tidy only once per header (PR #121436)

via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jan 1 05:30:17 PST 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 8e965d89c9624c184c48806dc39d50209265f0f8...3596a1f4217c26de0f237c8e2681c8574b6d17d3 libcxx/test/libcxx/clang_tidy.gen.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- clang_tidy.gen.py	2025-01-01 13:25:35.000000 +0000
+++ clang_tidy.gen.py	2025-01-01 13:29:52.913849 +0000
@@ -16,11 +16,12 @@
 import sys
 sys.path.append(sys.argv[1])
 from libcxx.header_information import lit_header_restrictions, lit_header_undeprecations, public_headers
 
 for header in public_headers:
-  print(f"""\
+    print(
+        f"""\
 //--- {header}.sh.cpp
 
 // REQUIRES: has-clang-tidy
 
 // The GCC compiler flags are not always compatible with clang-tidy.
@@ -34,6 +35,7 @@
 
 // TODO: run clang-tidy with modules enabled once they are supported
 // RUN: %{{clang-tidy}} %s --warnings-as-errors=* -header-filter=.* --config-file=%{{libcxx-dir}}/.clang-tidy --load=%{{test-tools-dir}}/clang_tidy_checks/libcxx-tidy.plugin -- -Wweak-vtables %{{compile_flags}} -fno-modules
 
 #include <{header}>
-""")
+"""
+    )

``````````

</details>


https://github.com/llvm/llvm-project/pull/121436


More information about the libcxx-commits mailing list