-
Notifications
You must be signed in to change notification settings - Fork 114
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
changelogs and version bump for foxy release #37
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.
There are several commits in master
, which should be merged before a ROS2 release.
Shouldn't we wait for #36 as well to be merged into the ROS2 branch?
* Update joint_state_publisher dependency (`#31 <https://github.com/ros-planning/moveit_resources/issues/31>`_) | ||
* Port to ROS 2 Eloquent (`#26 <https://github.com/ros-planning/moveit_resources/issues/26>`_) | ||
* Run Travis CI on ROS 2 Eloquent | ||
* Create moveit_resources.repos | ||
* Port CMakeLists.txt & package.xml to ROS2 | ||
* Travis: use ROS2 branch of moveit_ci (`#25 <https://github.com/ros-planning/moveit_resources/issues/25>`_) |
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.
#31 was just a fixup of #26. No need to explicitly list that in the changelogs.
* Update joint_state_publisher dependency (`#31 <https://github.com/ros-planning/moveit_resources/issues/31>`_) | |
* Port to ROS 2 Eloquent (`#26 <https://github.com/ros-planning/moveit_resources/issues/26>`_) | |
* Run Travis CI on ROS 2 Eloquent | |
* Create moveit_resources.repos | |
* Port CMakeLists.txt & package.xml to ROS2 | |
* Travis: use ROS2 branch of moveit_ci (`#25 <https://github.com/ros-planning/moveit_resources/issues/25>`_) | |
* Port and modernize CMakeLists.txt + package.xml to ROS2 (`#26 <https://github.com/ros-planning/moveit_resources/issues/26>`_, `#31 <https://github.com/ros-planning/moveit_resources/issues/31>`_) | |
* Travis: Port to ROS2 (`#25 <https://github.com/ros-planning/moveit_resources/issues/25>`_) |
@henningkayser what are your thoughts on waiting (and syncing) the upcoming changes in moveit_resources. |
I think it makes sense to wait for #36 and sync the changes, ideally we should try to be on the same level as the Noetic release. What are your thoughts on bumping the version to 2.0 similar to other ROS2 packages? |
No description provided.