refactor with pre-commit

Author: shenxianpeng

.gitignore

@@ -4,4 +4,4 @@ cpp_linter_hooks.egg-info
 cpp_linter_hooks/__pycache__/
 tests/.coverage
 tests/__pycache__
-.coverage
+.coverage

.pre-commit-config.yaml

@@ -1,4 +1,39 @@
 repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.3.0
+    hooks:
+      - id: trailing-whitespace
+        exclude: \.output
+      - id: end-of-file-fixer
+        exclude: \.(cp?p?$|output)
+      - id: check-docstring-first
+      - id: check-added-large-files
+      - id: check-yaml
+      - id: debug-statements
+      - id: requirements-txt-fixer
+  - repo: https://github.com/asottile/reorder_python_imports
+    rev: v3.7.0
+    hooks:
+      - id: reorder-python-imports
+        language_version: python3
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v1.16.1
+    hooks:
+      - id: pyupgrade
+  - repo: https://github.com/pycqa/flake8
+    rev: '4.0.1'
+    hooks:
+      - id: flake8
+        args: [--max-line-length=120]
+  - repo: local
+    hooks:
+      - id: tests
+        name: tests
+        entry: bash -c "pip install . && pytest -v tests"
+        description: Run all tests
+        language: system
+        types: [python]
+
   # Start: for development testing
   - repo: https://github.com/shenxianpeng/cpp-linter-hooks
     rev: latest
@@ -12,16 +47,3 @@ repos:
         # or specify the path of .clang-tidy
         # args: [--config-file=."]
   # End: for development testing
-  - repo: https://github.com/pycqa/flake8
-    rev: '4.0.1'
-    hooks:
-      - id: flake8
-        args: [--max-line-length=120]
-  - repo: local
-    hooks:
-      - id: tests
-        name: tests
-        entry: bash -c "pip install . && pytest -v tests"
-        description: Run all tests 
-        language: system
-        types: [python]

.pre-commit-hooks.yaml

@@ -25,7 +25,7 @@
 #   files: \.(h\+\+|h|hh|hxx|hpp|cuh|c|cc|cpp|cu|c\+\+|cxx|tpp|txx)$
 #   entry: clang-format -i
 #   args: ["-style=Google"]
-  
+
 # - id: docker-clang-tidy
 #   name: clang-tidy
 #   description: Run `clang-tidy` against C/C++ code to find warnings/errors in Docker container

README.md

@@ -16,7 +16,7 @@ repos:
     rev: v0.1.0  # Use the ref you want to point at
     hooks:
       - id: clang-format
-        args: [--style=Google] 
+        args: [--style=Google]
       # - id: clang-tidy ## Work in progress
       #   args: [--config-file=file]
 ```

cpp_linter_hooks/__init__.py

@@ -1,5 +1,7 @@
 import sys
-from cpp_linter_hooks.util import get_expect_version, check_installed
+
+from cpp_linter_hooks.util import check_installed
+from cpp_linter_hooks.util import get_expect_version
 
 
 clang_tools = ['clang-format', 'clang-tidy']

cpp_linter_hooks/clang_format.py

@@ -1,5 +1,7 @@
 import subprocess
-from cpp_linter_hooks import args, expect_version
+
+from cpp_linter_hooks import args
+from cpp_linter_hooks import expect_version
 
 
 def run_clang_format(args) -> int:

cpp_linter_hooks/clang_tidy.py

@@ -1,5 +1,7 @@
 import subprocess
-from cpp_linter_hooks import args, expect_version
+
+from cpp_linter_hooks import args
+from cpp_linter_hooks import expect_version
 
 
 def run_clang_tidy(args) -> int:

requirements-dev.txt

@@ -1,2 +1,2 @@
+coverage
 pytest
-coverage

setup.py

@@ -1,4 +1,5 @@
-from setuptools import setup, find_packages
+from setuptools import find_packages
+from setuptools import setup
 
 setup(
     name='cpp_linter_hooks',

testing/compile_flags.txt

@@ -1,2 +1,2 @@
 -Wall
--Werror
+-Werror