-
Notifications
You must be signed in to change notification settings - Fork 7
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
add WStrings message #4
Conversation
Signed-off-by: Dirk Thomas <[email protected]>
why is this part of test_msgs and not a built-in type? https://index.ros.org/doc/ros2/Concepts/About-ROS-Interfaces/#field-types |
This PR introduces a message for testing the built-in type. It looks like the ROS Index page is missing info on |
Shouldn't it be in std_msgs too to be considered a built-in type? |
@adamsj-oci there is a difference between built-in types and messages. A built-in type can be used as a field type in a message but is itself not a message. |
@dirk-thomas That makes sense but at the same time it seems logical to have a |
I agree that it makes sense to add a |
This package as the name indicates is only for tests. You should create your own message definition to give the interface semantic meaning - for the same reason why you shouldn't use |
Connect to ros2/rosidl#352.