chore(pre-commit): replace ament linters with g++ warning-based linting

Removed ament_cppcheck and ament_cpplint hooks, added gcc-warnings hook with arguments for C/C++ files.

Author: kluge7

.pre-commit-config.yaml

@@ -76,34 +76,12 @@ repos:
         args: [--style=file]
   - repo: local
     hooks:
-      - id: ament_cppcheck
-        name: ament_cppcheck
-        description: Static code analysis of C/C++ files.
-        entry: env AMENT_CPPCHECK_ALLOW_SLOW_VERSIONS=1 ament_cppcheck
+      - id: gcc-warnings
+        name: gcc-warningss
         language: system
-        files: \.(h\+\+|h|hh|hxx|hpp|cuh|c|cc|cpp|cu|c\+\+|cxx|tpp|txx)$
-        exclude: (^|/)(test|tests)/.*  # exclude test dirs since ament_cppcheck misparses GTest macros and throws false syntax errors
-  - repo: local
-    hooks:
-      - id: ament_cpplint
-        name: ament_cpplint
-        description: Static code analysis of C/C++ files.
-        entry: ament_cpplint
-        language: system
-        files: \.(h\+\+|h|hh|hxx|hpp|cuh|c|cc|cpp|cu|c\+\+|cxx|tpp|txx)$
-        args: [
-          "--linelength=120",
-          "--filter=-whitespace/newline,-legal/copyright,-build/include_order" # ignore build/include_order since it conflicts with .clang-format
-        ]
-  # CMake hooks
-  - repo: local
-    hooks:
-      - id: ament_lint_cmake
-        name: ament_lint_cmake
-        description: Check format of CMakeLists.txt files.
-        entry: ament_lint_cmake
-        language: system
-        files: CMakeLists\.txt$
+        entry: g++
+        args: ["-Wall", "-Wextra", "-pedantic", "-Werror", "-fsyntax-only"]
+        files: \.(c|cc|cpp|cxx|h|hpp)$
   # Spellcheck in comments and docs
   - repo: https://github.com/codespell-project/codespell
     rev: v2.4.1