[libcxx] [llvm] [libc++][modules] Adds module testing. (PR #76246)

via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 22 08:08:11 PST 2023


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 18170d0f281c2cae5252c501cbcd174783de78b7...03967271598508aec80804278f8235cb9056a832 libcxx/test/libcxx/module_std.gen.py libcxx/test/libcxx/module_std_compat.gen.py libcxx/utils/libcxx/test/features.py libcxx/utils/libcxx/test/format.py libcxx/utils/libcxx/test/modules.py
``````````

</details>

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

``````````diff
--- utils/libcxx/test/format.py	2023-12-22 16:04:20.000000 +0000
+++ utils/libcxx/test/format.py	2023-12-22 16:08:03.627158 +0000
@@ -144,13 +144,16 @@
         if module == "std":
             has_std_module = True
         elif module == "std.compat":
             has_std_compat_module = True
         else:
-             script.insert(0, f"echo \"The module '{module}' is not valid, use 'std' or 'std.compat'\"")
-             script.insert(1, "false");
-             return script
+            script.insert(
+                0,
+                f"echo \"The module '{module}' is not valid, use 'std' or 'std.compat'\"",
+            )
+            script.insert(1, "false")
+            return script
 
     if modules:
         # This flag is needed for both modules.
         moduleCompileFlags.append("-fprebuilt-module-path=%T")
 

``````````

</details>


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


More information about the llvm-commits mailing list