-
Notifications
You must be signed in to change notification settings - Fork 52
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
adding a test and a commented out test for the bind that doesn't compile #88
Conversation
@esteve For full testing of ros2/rclcpp#183 you'll need to uncomment the second added test block in this branch. |
I got this to compile fine together with ros2/rclcpp#183, at least locally on Linux |
I've merged ros2/rclcpp#183, so if you uncomment the test here, I think the CI jobs will pass. |
I'm testing it locally against your branch. If it passes I'll update the PR |
8d8a2cb
to
7547dee
Compare
+1 |
7547dee
to
c20352f
Compare
0c85fbe
to
41964e7
Compare
4439e4b
to
ce6dc44
Compare
@tfoote I've added support for plain functions and |
Thanks @esteve it's not necessary for my use case but I suspect that it will be useful to others. |
adding a test and a commented out test for the bind that doesn't compile
Connects to ros2/rclcpp#173