Fix clang-10 formatting check (faasm#282)

* format: fix clang-10 formatting check * apply pr comments
auto-ndp · Nov 7, 2022 · 18f98f2 · 18f98f2
1 parent 3d17b93
commit 18f98f2
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/tasks/format_code.py b/tasks/format_code.py
@@ -51,7 +51,12 @@ def format(ctx, check=False):
         .split("\n")[:-1]
     )
 
-    clang_cmd = "clang-format-10 -i {}".format(" ".join(files_to_check))
+    clang_cmd = [
+        "clang-format-10",
+        "--dry-run --Werror" if check else "-i",
+        " ".join(files_to_check),
+    ]
+    clang_cmd = " ".join(clang_cmd)
     run(clang_cmd, shell=True, check=True, cwd=PROJ_ROOT)
 
     # ---- Append newlines to C/C++ files if not there ----