update readme and debug clang-tidy

Author: shenxianpeng

.pre-commit-config.yaml

@@ -17,7 +17,7 @@ repos:
       - id: reorder-python-imports
         language_version: python3
   - repo: https://github.com/asottile/pyupgrade
-    rev: v1.16.1
+    rev: v2.36.0
     hooks:
       - id: pyupgrade
   - repo: https://github.com/pycqa/flake8
@@ -39,9 +39,9 @@ repos:
     rev: main
     hooks:
       - id: clang-format
-        args: [--style=Google, --version=14]
+        args: [--style=Google, --version=13]
       - id: clang-tidy
-        args: [--checks="boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*"]
+        args: [--checks=testing/.clang-tidy, --version=13]
         # or specify the path of .clang-tidy
         # args: [--config-file=."]
   # End: for development testing

README.md

@@ -17,8 +17,21 @@ repos:
     hooks:
       - id: clang-format
         args: [--style=Google]
-      # - id: clang-tidy ## Work in progress
-      #   args: [--config-file=file]
+      - id: clang-tidy
+        args: [--checks='boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*']
+```
+
+Use specific version. for example `clang-format` is version 13, `clang-tidy` is version 12.
+
+```yaml
+repos:
+  - repo: https://github.com/shenxianpeng/cpp-linter-hooks
+    rev: v0.1.0  # Use the ref you want to point at
+    hooks:
+      - id: clang-format
+        args: [--style=Google, --version=13]
+      - id: clang-tidy
+        args: [--checks='boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*', --version=12]
 ```
 
 ## Support hooks
@@ -30,6 +43,31 @@ Prevent committing unformatted C/C++ code.
 * Set coding style: LLVM, GNU, Google, Chromium, Microsoft, Mozilla, WebKit with `args: [--style=LLVM]`
 * Load coding style configuration file `.clang-format` with `args: [--style=file]`
 
+output
+
+```
+clang-format.............................................................Failed
+- hook id: clang-format
+- files were modified by this hook
+```
+modified file
+```diff
+--- a/testing/main.c
++++ b/testing/main.c
+@@ -1,3 +1,6 @@
+ #include <stdio.h>
+-int main() {for (;;) break; printf("Hello world!\n");return 0;}
+-
++int main() {
++  for (;;) break;
++  printf("Hello world!\n");
++  return 0;
++}
+```
+
 ### `clang-tidy`
 
 Prevent committing typical programming errors, like style violations, interface misuse, or bugs that can be deduced.
+
+* 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]`

cpp_linter_hooks/clang_tidy.py

@@ -13,6 +13,7 @@ def run_clang_tidy(args) -> int:
         if arg == expect_version or arg.startswith("--version"):
             continue
         command.append(arg)
+    print(command)
     try:
         subprocess.run(command, stdout=subprocess.PIPE)
         return 0