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 all references to "MoveIt" to include exclamation mark e.g. "MoveIt!" #1020
Fix all references to "MoveIt" to include exclamation mark e.g. "MoveIt!" #1020
Conversation
This PR also applies clang format on the |
Somehow you ran the wrong type of clang-format - it should not have caused this many changes. Can you remove the "apply clang format" commit and try again using the instructions here: http://moveit.ros.org/documentation/contributing/code/ Maybe you have the wrong version of clang-format? |
Yes I've noticed that |
a31b489
to
b149204
Compare
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.
Looks good. Two nits on formatting.
structure describes that controller's state. */ | ||
struct ControllerState | ||
{ | ||
ControllerState() : active_(false), default_(false) | ||
{ | ||
} | ||
|
||
/** \brief A controller can be active or inactive. This means that MoveIt could activate the controller when needed, | ||
/** \brief A controller can be active or inactive. This means that MoveIt! could activate the controller when | ||
needed, |
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.
Fix wrapping with line below
and de-activate controllers that overlap (control the same set of joints) */ | ||
bool active_; | ||
|
||
/** \brief It is often the case that multiple controllers could be used to execute a motion. Marking a controller as | ||
default | ||
makes MoveIt prefer this controller when multiple options are available. */ | ||
makes MoveIt! prefer this controller when multiple options are available. */ |
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.
text wrapping
Fixed text wrap 👍 @mcevoyandy |
b149204
to
0f3229b
Compare
cherry-pick: #1030 |
Description
Fix all references to "MoveIt" to include exclamation mark e.g. "MoveIt!".
Checklist