ci(config): add missing CPPLINT.cfg (autowarefoundation#47)

* Add settings for new formatters (autowarefoundation#2330) * Update .pre-commit-config.yaml Signed-off-by: Kenji Miyake <[email protected]> * Ignore .cu for cpplint Signed-off-by: Kenji Miyake <[email protected]> * Update .clang-format Signed-off-by: Kenji Miyake <[email protected]> * Add setup.cfg Signed-off-by: Kenji Miyake <[email protected]> * Add CPPLINT.cfg Signed-off-by: Kenji Miyake <[email protected]> * Add autoware-lint-common Signed-off-by: Kenji Miyake <[email protected]> * Remove unnecessary part Signed-off-by: Kenji Miyake <[email protected]> * Add includeorder=standardcfirst Signed-off-by: Kenji Miyake <[email protected]> * Use clang-format-12 Signed-off-by: Kenji Miyake <[email protected]> * Add IncludeCategories to clang-format Signed-off-by: Kenji Miyake <[email protected]> * Remove unnecessary file Signed-off-by: Kenji Miyake <[email protected]> * Temporarily comment out pre-commit hooks Signed-off-by: Kenji Miyake <[email protected]> * Fix indent miss Signed-off-by: Kenji Miyake <[email protected]> * Change include order rule and ignore build/include_order in cpplint Signed-off-by: Kenji Miyake <[email protected]> * ci(pre-commit): autofix Co-authored-by: Kenji Miyake <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
satoshi-ota · Nov 30, 2021 · 57da9a8 · 57da9a8
1 parent 57900c6
commit 57da9a8
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -13,4 +13,3 @@ log/
 
 # Python
 *.pyc
-
diff --git a/CPPLINT.cfg b/CPPLINT.cfg
@@ -0,0 +1,13 @@
+# Modified from https://github.com/ament/ament_lint/blob/ebd524bb9973d5ec1dc48a670ce54f958a5a0243/ament_cpplint/ament_cpplint/main.py#L64-L120
+set noparent
+linelength=100
+includeorder=standardcfirst
+filter=-build/c++11               # we do allow C++11
+filter=-build/namespaces_literals # we allow using namespace for literals
+filter=-runtime/references        # we consider passing non-const references to be ok
+filter=-whitespace/braces         # we wrap open curly braces for namespaces, classes and functions
+filter=-whitespace/indent         # we don't indent keywords like public, protected and private with one space
+filter=-whitespace/parens         # we allow closing parenthesis to be on the next line
+filter=-whitespace/semicolon      # we allow the developer to decide about whitespace after a semicolon
+filter=-build/header_guard        # TODO(Kenji Miyake): Support ROS-style rule in cpplint or add auto-fix script in pre-commit
+filter=-build/include_order       # we use the custom include order