Skip to content
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

fix typo getCollisionBodyTransform(s) #1107

Merged
merged 3 commits into from Oct 27, 2018
Merged

Conversation

rhaschke
Copy link
Contributor

This method only returns a single transform, hence a singular form name should be used.
Interestingly this method was never used inside MoveIt.

@v4hn
Copy link
Contributor

v4hn commented Oct 23, 2018

I don't really see the need but fine by me as long as this is not backported to kinetic.

Please add the change to MIGRATION.md before merging.

@rhaschke
Copy link
Contributor Author

I agree, there is no particular need for this change. Just stumbled over it.

MIGRATION.md Outdated Show resolved Hide resolved
Co-Authored-By: rhaschke <rhaschke@techfak.uni-bielefeld.de>
@rhaschke rhaschke merged commit e4a3961 into moveit:melodic-devel Oct 27, 2018
@rhaschke rhaschke deleted the fix-typo branch October 27, 2018 04:18
JafarAbdi pushed a commit to JafarAbdi/moveit that referenced this pull request Mar 24, 2022
* Add galactic CI
* Comment out rolling
* panda_ros_controllers -> panda_ros2_controllers
* Ignore flake8 tests
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.

None yet

3 participants