update clang-tidy

Author: shenxianpeng

.pre-commit-config.yaml

@@ -36,12 +36,11 @@ repos:
 
   # Start: for development testing
   - repo: https://github.com/shenxianpeng/cpp-linter-hooks
-    rev: 39b0aaa721fb0b1d6fb07214e6bfa63322419e5a
+    rev: cf62613fe77f66cc826cbebcf2693bf0872dc764
     hooks:
       - id: clang-format
         args: [--style=Google, --version=13]
       - id: clang-tidy
-        verbose: true
         args: [--checks=testing/.clang-tidy, --version=13]
         # or specify the path of .clang-tidy
         # args: [--config-file=."]

README.md

@@ -71,3 +71,25 @@ Prevent committing typical programming errors, like style violations, interface
 
 * Set checks like `args: [--checks='boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*']`
 * Or set specify the path of .clang-tidy like `args: [--checks=path/to/.clang-tidy]`
+
+
+Output
+
+```
+clang-tidy...............................................................Failed
+- hook id: clang-tidy
+- duration: 0.48s
+
+418 warnings and 1 error generated.
+Error while processing /home/ubuntu/cpp-linter-hooks/testing/main.c.
+Suppressed 417 warnings (417 in non-user code).
+Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
+Found compiler error(s).
+/home/ubuntu/cpp-linter-hooks/testing/main.c:3:11: warning: statement should be inside braces [readability-braces-around-statements]
+  for (;;) break;
+          ^
+           {
+/usr/include/stdio.h:33:10: error: 'stddef.h' file not found [clang-diagnostic-error]
+#include <stddef.h>
+         ^~~~~~~~~~
+```

cpp_linter_hooks/clang_tidy.py

@@ -4,7 +4,7 @@
 from cpp_linter_hooks import expect_version
 
 
-def run_clang_tidy(args) -> str:
+def run_clang_tidy(args) -> int:
     if expect_version:
         command = [f'clang-tidy-{expect_version}']
     else:
@@ -14,15 +14,18 @@ def run_clang_tidy(args) -> str:
             continue
         command.append(arg)
     try:
-        output = subprocess.run(command, stdout=subprocess.PIPE).stdout.decode("utf-8")
+        sp = subprocess.run(command, stdout=subprocess.PIPE)
+        retval = sp.returncode
+        output = sp.stdout.decode("utf-8")
+        return retval, output
     except FileNotFoundError as e:
-        output = e
-    return output
+        return 1, e
 
 
 def main():
-    result = run_clang_tidy(args)
-    print(result)
+    retval, output = run_clang_tidy(args)
+    if retval != 0:
+        print(output)
 
 
 if __name__ == "__main__":