Skip to content

moveit_experimental: commenting out dependencies#1256

Merged
v4hn merged 1 commit intomoveit:melodic-develfrom
ubi-agni:cleanup-experimental
Dec 12, 2018
Merged

moveit_experimental: commenting out dependencies#1256
v4hn merged 1 commit intomoveit:melodic-develfrom
ubi-agni:cleanup-experimental

Conversation

@rhaschke
Copy link
Copy Markdown
Contributor

As suggested in #1251 (comment), comment-out all stuff in moveit_experimental, as there is nothing built.

Currently, there is nothing built in moveit_experimental
<author email="jnicho@swri.org">Jorge Nicho</author>

<maintainer email="mferguson@fetchrobotics.com">Michael Ferguson</maintainer>
<maintainer email="chitt@live.in">Chittaranjan Srinivas Swaminathan</maintainer>
Copy link
Copy Markdown
Member

@gavanderhoorn gavanderhoorn Dec 11, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that manifests without any maintainer tags are invalid and will result in catkin erroring out.

sorry, missed the single maintainer below.

@v4hn
Copy link
Copy Markdown
Contributor

v4hn commented Dec 12, 2018

I proposed this change to begin with, so I approve.
Thanks!

@v4hn v4hn merged commit 1674212 into moveit:melodic-devel Dec 12, 2018
@rhaschke rhaschke deleted the cleanup-experimental branch December 13, 2018 19:15
v4hn pushed a commit that referenced this pull request Dec 21, 2018
Currently, there is nothing built in moveit_experimental
v4hn pushed a commit that referenced this pull request Dec 21, 2018
Currently, there is nothing built in moveit_experimental
rhaschke added a commit that referenced this pull request Jan 1, 2019
Currently, there is nothing built in moveit_experimental
pull bot pushed a commit to shadow-robot/moveit that referenced this pull request Sep 3, 2020
Currently, there is nothing built in moveit_experimental
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants