-
Notifications
You must be signed in to change notification settings - Fork 128
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
Release moveit_msgs (I 0.7.2, J 0.8.1) #22
Comments
Ok, I'm running prerelease tests for I & J locally with 2 level downstream. I'm sure it'll take time. |
130s: good point mentioning jade, I just cherry-picked the changes to moveit_msgs and moveit_core to their respective jade-devel branches. (I suppose it's valid to cherry-pick commits that have been reviewed to other branches unless they might break something there?) Probably too late for your tests though @130s |
@v4hn no worries, I just re-triggerred the script (2 commands). Note that now I have a slight suspicion how effective ROS Prerelease test given the issue ros-infrastructure/prerelease_website#31. We may or may not need to pay too much attention to the result of my Prerelease Test for now. |
@v4hn feel free to cherry pick to newer branches without review, yes. but lets PR backports |
Indigo passed. Jade prerelease test fails at downstream (log pasted below). I am +1 for making a release since this failing one is a leaf.
|
+1 |
Ok, I'm on this for I&J. |
@mikeferguson @IanTheEngineer @130s Can we do a release of moveit_msgs in indigo for this new feature?
The text was updated successfully, but these errors were encountered: