Merge pull request #4 from jgaffiot/testing

Testing

Author: jgaffiot

.flake8

@@ -8,16 +8,13 @@ exclude =
     build,
     install,
     .pytest_cache,
+    pkg_templates/*,
 
 # it's not a bug that we aren't using all of hacking
 ignore =
-    # H104: file contains nothing but comments
-    H104
-    # H301: one import per line
-    H301,
-    # H306: imports not in alphabetical order (time, os)
-    H306,
-    # W503: line break before binary operator
-    W503
-    # E203: whitespace before ':', because of slicing (such as list[2 : 3])
-    E203
+    H104, # file contains nothing but comments (OK for __init__.py)
+    H301, # one import per line (too restrictive)
+    H306, # imports not in alphabetical order (time, os) (boring and of little use)
+    W503, # line break before binary operator (useless with black)
+    E203, # whitespace before ':' (useless with black)
+    S101  # use of assert detected (useless with pytest)

.gitignore

@@ -39,3 +39,7 @@ install/*
 *.exe
 *.out
 *.app
+
+# Python cache
+__pycache__/
+*.pyc

.pre-commit-config.yaml

@@ -25,14 +25,6 @@ repos:
         args: [--py38-plus]
 
 # C++
-      - id: compilation
-        name: Compilation
-        description: Compile the project
-        language: system
-        always_run: true
-        entry: make all
-        files: Makefile
-
       - id: clang-format
         name: Clang-format
         description: Run `clang-format` against C/C++ header and source files
@@ -55,6 +47,14 @@ repos:
         entry: cppcheck --error-exitcode=1
         args: [--enable=warning]
 
+      - id: compilation
+        name: Compilation
+        description: Compile the project
+        language: system
+        always_run: true
+        entry: make -j all
+        files: Makefile
+
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.0.1
     hooks:
@@ -100,8 +100,6 @@ repos:
         types: [python]
       - id: debug-statements
         types: [python]
-      - id: fix-encoding-pragma
-        types: [python]
       - id: requirements-txt-fixer
 
   - repo: https://github.com/jumanjihouse/pre-commit-hook-yamlfmt

CPPLINT.cfg

@@ -8,5 +8,6 @@ filter=-whitespace/parens
 filter=-whitespace/braces
 filter=-whitespace/indent
 filter=-whitespace/operators
+filter=-whitespace/semicolon
 filter=-readability/alt_tokens
 filter=-runtime/references

Makefile

@@ -2,12 +2,14 @@ RELEASE_DIR := ./build/release/
 DEBUG_DIR   := ./build/debug/
 
 TOOLS_DIR := $(dir $(abspath $(lastword $(MAKEFILE_LIST))))
-MAKE += --no-print-directory
-CMAKE := cmake $(TOOLS_DIR)/source #--trace --debug-output
+CMAKE := cmake $(TOOLS_DIR)/source
 FLAG := -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
 
-ifdef DEBUG
-	FLAG += -DDEBUG=$(DEBUG)
+ifdef DEBUG_BUILD
+	CMAKE += --trace --debug-output
+	MAKE += --trace -d
+else
+	MAKE += --no-print-directory
 endif
 
 all: install_release