Skip to content

Add developer tooling: .gitignore, .clang-format, pre-commit#361

Merged
rsasaki0109 merged 1 commit intodevelop-ros2from
feature/dev-tooling
Apr 6, 2026
Merged

Add developer tooling: .gitignore, .clang-format, pre-commit#361
rsasaki0109 merged 1 commit intodevelop-ros2from
feature/dev-tooling

Conversation

@rsasaki0109
Copy link
Copy Markdown
Collaborator

Summary

  • Expand .gitignore to cover ROS workspace artifacts (build/, install/, log/), compiled binaries, IDE files, and OS files
  • Add .clang-format based on Google style (C++17, column limit 100)
  • Add .pre-commit-config.yaml with hooks for trailing whitespace, clang-format, and cmake-format

How to set up pre-commit locally

pip install pre-commit
pre-commit install

Copy link
Copy Markdown

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 To request another review, post a new comment with "/windsurf-review".

PenaltyReturnTypeOnItsOwnLine: 200
PointerAlignment: Left
ReflowComments: false
SortIncludes: false
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider setting SortIncludes: true to automatically organize include statements. This is generally considered a best practice and would be more consistent with the Google style guide that this configuration is based on.

@rsasaki0109 rsasaki0109 merged commit 7991830 into develop-ros2 Apr 6, 2026
2 checks passed
@rsasaki0109 rsasaki0109 deleted the feature/dev-tooling branch April 6, 2026 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant