Use MoveIt! clang format for SRDF

moveit / srdfdom

Semantic Robot Description Format

BSD 3-Clause "New" or "Revised" License

13 stars 68 forks source link

Use MoveIt! clang format for SRDF #42

Closed davetcoleman closed 6 years ago

davetcoleman commented 6 years ago

@kkufieta

jonbinney commented 6 years ago

Looks good - I'm building this just to sanity check.

rhaschke commented 6 years ago

@jonbinney Please be more careful before merging. This PR was failing on Travis!

jonbinney commented 6 years ago

@rhaschke my mistake - will be more careful in the future.

To clarify, this didn't fail any of the existing checks. It added a new check that fails right now on melodic, since clang-format-3.8 package doesn't exist on 18.04. That should be fixed once https://github.com/ros-planning/moveit_ci/pull/39 is debugged and merged.