Skip to content

Commit 57da9a8

Browse files
taikitanaka3kenji-miyakepre-commit-ci[bot]
authored
ci(config): add missing CPPLINT.cfg (#47)
* Add settings for new formatters (#2330) * Update .pre-commit-config.yaml Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Ignore .cu for cpplint Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Update .clang-format Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Add setup.cfg Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Add CPPLINT.cfg Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Add autoware-lint-common Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Remove unnecessary part Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Add includeorder=standardcfirst Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Use clang-format-12 Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Add IncludeCategories to clang-format Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Remove unnecessary file Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Temporarily comment out pre-commit hooks Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Fix indent miss Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * Change include order rule and ignore build/include_order in cpplint Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> * ci(pre-commit): autofix Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 57900c6 commit 57da9a8

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

.gitignore

-1
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,3 @@ log/
1313

1414
# Python
1515
*.pyc
16-

CPPLINT.cfg

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

0 commit comments

Comments
 (0)