Are hyphens allowed in ROS 2 package names?
In ROS 1 they used to be forbidden but are now allowed (https://github.com/ros-infrastructure... and connected links)
In ROS 2 it used to be forbidden (for catkin/ament packages):
ament_package.exceptions.InvalidPackage: Invalid package manifest './src/foo/package.xml': Package name 'foo-bar' does not follow naming conventions
But seems to not be enforced since the switch to colcon/catkin_pkg:
root@76e629018ac5:/ws# colcon build
Starting >>> foo-bar
Finished <<< foo-bar [1.41s]
Summary: 1 package finished [1.51s]
Does this mean dashes are now allowed in ROS 2 package names? If yes, is there any recommendations about using/not using them ?
Not an answer (obviously), but: bah, dashes in package names. Note @Dirk Thomas' comment on the PR you reference:
Agreed and I hope it won't be used for ROS packages either (neither ROS 1 nor ROS 2).