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

Rename jog_arm->moveit_jog_arm #1663

Merged
merged 3 commits into from
Sep 19, 2019
Merged

Rename jog_arm->moveit_jog_arm #1663

merged 3 commits into from
Sep 19, 2019

Conversation

AndyZe
Copy link
Member

@AndyZe AndyZe commented Sep 9, 2019

Closes #1612

Copy link
Contributor

@rhaschke rhaschke left a comment

Choose a reason for hiding this comment

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

That's a lot of renaming. Take the chance to clean up a little bit as well. See suggestions below.

Copy link
Contributor

@rhaschke rhaschke left a comment

Choose a reason for hiding this comment

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

Thanks for the cleanup. I fixed clang-format issues. If Travis is happy then, I am as well.

@rhaschke rhaschke added the awaits 2nd review one maintainer approved this request label Sep 18, 2019
Copy link
Member

@davetcoleman davetcoleman left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Note you don't need my second review to merge this as @AndyZe is a Core Contributor on the webpage:

Quicker code contributions: you only need one review to get your code merged in, while non-maintainer/non-core contributor pull requests require two approvals.

@davetcoleman davetcoleman merged commit ff71fc4 into moveit:master Sep 19, 2019
henningkayser pushed a commit to PickNikRobotics/moveit that referenced this pull request Nov 21, 2019
* Rename jog_arm->moveit_jog_arm

* Address @rhaschke's feedback

* apply clang-format
@tylerjw tylerjw mentioned this pull request May 8, 2020
20 tasks
tylerjw pushed a commit to PickNikRobotics/moveit that referenced this pull request May 12, 2020
* Rename jog_arm->moveit_jog_arm

* Address @rhaschke's feedback

* apply clang-format
tylerjw pushed a commit to PickNikRobotics/moveit that referenced this pull request May 12, 2020
* Rename jog_arm->moveit_jog_arm

* Address @rhaschke's feedback

* apply clang-format
tylerjw pushed a commit to PickNikRobotics/moveit that referenced this pull request May 20, 2020
* Rename jog_arm->moveit_jog_arm

* Address @rhaschke's feedback

* apply clang-format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaits 2nd review one maintainer approved this request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rename jog_arm to moveit_jog_arm
3 participants