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

updated to use node getter functions #329

Merged
merged 4 commits into from
Apr 25, 2022

Conversation

jackcenter
Copy link

@jackcenter jackcenter commented Apr 13, 2022

Solves ros2_control/issues/679

Depends on: ros2_control/pull/699

Replaced node_ everywhere with get_node() and with get_node_const() inside const functions.

Copy link
Member

@destogl destogl left a comment

Choose a reason for hiding this comment

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

In general, it is good, but let's check this const thing (see comment)

@bmagyar bmagyar merged commit 53d3b25 into ros-controls:master Apr 25, 2022
destogl added a commit to StoglRobotics-forks/ros2_controllers that referenced this pull request Apr 26, 2022
* updated to use node getter functions

* updated get_node_const() name

Co-authored-by: Denis Štogl <denis@stogl.de>
Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
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