-
Notifications
You must be signed in to change notification settings - Fork 240
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 API samples on main branch - Rolling C++ API examples #1068
Conversation
Hey @gbiggs @MichaelOrlov what do you think about this? Any concern merging it? It's mostly a direct copy but just moved directories/package names a little bit to feel closer to ros2/examples |
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.
@emersonknapp Thanks for iterating on this PR.
I would be glad to merge this PR, although I found missing create_topic
and usage of the regular subscription instead of generic_subscription
.
Just realized this wasn't in the Action CI - so I've added that, and if it all succeeds I'll run Jenkins CI |
Gist: https://gist.githubusercontent.com/emersonknapp/a83d5388f1b35e6e3c20302a1c9d8a58/raw/a820f6e494d6d8444ade9a1f5614ff5d86e57e38/ros2.repos |
@ros-pull-request-builder retest this please |
Warning on Windows build supposed to be already fixed with #1084 |
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
b549296
to
64194c9
Compare
Port to latest for #869 - so that it can be kept up to date and automatically included with new Releases, rather than having to be added after the fact to distros.