Skip to content

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

Open
rsasaki0109 wants to merge 1 commit intodevelop-ros2from
feature/dev-tooling
Open

Add developer tooling: .gitignore, .clang-format, pre-commit#361
rsasaki0109 wants to merge 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.

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