-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change UUID type in action_msgs #54
Change UUID type in action_msgs #54
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, pending green CI
cacc2c8
to
c84b7cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with green CI if the IDL PRs don't get merged in time. Actually this one doesn't depend on the IDL prs, oops.
CI with this and ros2/rcl#338 (not including ros2/rosidl#332 since that should be merged after ongoing IDL work) |
* Change UUID type in action_msgs * Added unique_identifier_msgs dependency to action_msgs
This PR is part of a series of three that together close #49
EDIT: Depends on ros2/ros2#609
uint8[16]
field from GoalInfo.msg withunique_identifier_msgs/uuid