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

Commit

Permalink
Merge pull request #70 from SSL-Roots/update_contributing
Browse files Browse the repository at this point in the history
Add an explanation about format check in CONTRIBUTING.md
  • Loading branch information
nabekuro0104 authored Jul 19, 2020
2 parents 37bd1c2 + a86e20b commit 4a2b4e3
Showing 1 changed file with 29 additions and 2 deletions.
31 changes: 29 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,36 @@ issueの内容については制限を設けていません。
[ROS 2 Developer Guide](https://index.ros.org/doc/ros2/Contributing/Developer-Guide/)
に従います。

将来的にはlintを使ってチェックする予定です。
ファイルのフォーマットは
[ament_lint](https://github.com/ament/ament_lint)
によってビルド時にチェックされます。

ローカルでフォーマットをチェックする場合は次のコマンドを実行します。

```sh
$ cd ~/ros2_ws
$ colcon test
# テストの結果を表示
$ colcon test-result --verbose
```

また、次のようにファイルを個別にチェックすることも可能です。

```sh
# C++のコードはament_uncrustifyやament_clang_formatによってフォーマットをチェックできます
$ cd ~/ros2_ws/src/consai2r2
# チェック結果を表示
$ ament_uncrustify consai2r2_teleop/src/joystick_component.cpp
# フォーマットを修正(ファイルの中身が書き換わります)
$ ament_uncrustify --reformat consai2r2_teleop/src/joystick_component.cpp
```

`consai2r2`のホームディレクトリには[clang-formatの設定ファイル](./.clang-format)を用意しています。
clang-formatコマンドや、エディタのフォーマッタ設定等に活用してください。

`.clang-format`ファイルを使用しても`colcon test`に失敗する場合があります。
**[GitHub Actions](https://github.com/SSL-Roots/consai2r2/actions)でのテストが通るように修正してください。**

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

## その他

Expand Down

0 comments on commit 4a2b4e3

Please sign in to comment.