Skip to content
This repository was archived by the owner on Jan 28, 2023. It is now read-only.

Commit 4a2b4e3

Browse files
authored
Merge pull request #70 from SSL-Roots/update_contributing
Add an explanation about format check in CONTRIBUTING.md
2 parents 37bd1c2 + a86e20b commit 4a2b4e3

1 file changed

Lines changed: 29 additions & 2 deletions

File tree

CONTRIBUTING.md

Lines changed: 29 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,36 @@ issueの内容については制限を設けていません。
2929
[ROS 2 Developer Guide](https://index.ros.org/doc/ros2/Contributing/Developer-Guide/)
3030
に従います。
3131

32-
将来的にはlintを使ってチェックする予定です。
32+
ファイルのフォーマットは
33+
[ament_lint](https://github.com/ament/ament_lint)
34+
によってビルド時にチェックされます。
35+
36+
ローカルでフォーマットをチェックする場合は次のコマンドを実行します。
37+
38+
```sh
39+
$ cd ~/ros2_ws
40+
$ colcon test
41+
# テストの結果を表示
42+
$ colcon test-result --verbose
43+
```
44+
45+
また、次のようにファイルを個別にチェックすることも可能です。
46+
47+
```sh
48+
# C++のコードはament_uncrustifyやament_clang_formatによってフォーマットをチェックできます
49+
$ cd ~/ros2_ws/src/consai2r2
50+
# チェック結果を表示
51+
$ ament_uncrustify consai2r2_teleop/src/joystick_component.cpp
52+
# フォーマットを修正(ファイルの中身が書き換わります)
53+
$ ament_uncrustify --reformat consai2r2_teleop/src/joystick_component.cpp
54+
```
55+
56+
`consai2r2`のホームディレクトリには[clang-formatの設定ファイル](./.clang-format)を用意しています。
57+
clang-formatコマンドや、エディタのフォーマッタ設定等に活用してください。
58+
59+
`.clang-format`ファイルを使用しても`colcon test`に失敗する場合があります。
60+
**[GitHub Actions](https://github.com/SSL-Roots/consai2r2/actions)でのテストが通るように修正してください。**
3361

34-
その他の細かいルールが決まり次第、本ファイルに追記します。
3562

3663
## その他
3764

0 commit comments

Comments
 (0)