-
Notifications
You must be signed in to change notification settings - Fork 193
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
Add documentation on node parameter #263
Conversation
Update the documentation to indicate that the `__name` parameter provides the same functionality as `__node`. Also fix some incorrect references to ROS1 behavior. Related to ros2/rcl#258.
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.
LGTM but for a nit.
Co-Authored-By: Michel Hidalgo <michel@ekumenlabs.com>
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.
LGTM
@hidmic Can this be merged now that the related ticket has been resolved? |
@hidmic bump |
Thanks for the bump guys, and apologies for the delay. Thank you for your contribution @pjreed! |
* Add documentation on node parameter Update the documentation to indicate that the `__name` parameter provides the same functionality as `__node`. Also fix some incorrect references to ROS1 behavior. Related to ros2/rcl#258. * Update articles/141_static_remapping.md Co-Authored-By: Michel Hidalgo <michel@ekumenlabs.com> Co-authored-by: Michel Hidalgo <michel@ekumenlabs.com>
* Add documentation on node parameter Update the documentation to indicate that the `__name` parameter provides the same functionality as `__node`. Also fix some incorrect references to ROS1 behavior. Related to ros2/rcl#258. * Update articles/141_static_remapping.md Co-Authored-By: Michel Hidalgo <michel@ekumenlabs.com> Co-authored-by: Michel Hidalgo <michel@ekumenlabs.com>
Update the documentation to indicate that the
__name
parameter provides the same functionality as__node
. Also fix some incorrect references to ROS1 behavior.Related to ros2/rcl#258.