search

ros-planning / navigation_msgs

Message packages required by the navigation stack
42 stars 53 forks source link

add missing separator #5

Closed wjwwood closed 6 years ago

wjwwood commented 6 years ago

Some of the .srv files are missing the --- if there is no response content, but ROS 2 requires that to be there.

See: https://github.com/ros2/rviz/pull/267#issuecomment-396839299

Connects to ros2/rviz#99

wjwwood commented 6 years ago

@mikaelarguedas I went ahead and made a pull request for this.

mikaelarguedas commented 6 years ago

:+1:

I dont have power in this organization (and should probably not approve my own PRs anyway) but the change looks good to me :)

mikeferguson commented 6 years ago

LGTM