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

Dashing update #47

Merged
merged 7 commits into from
Feb 6, 2020
Merged

Dashing update #47

merged 7 commits into from
Feb 6, 2020

Conversation

Karsten1987
Copy link
Contributor

connects to ros-controls/ros2_controllers#7
backport of the current master branch to be on pair with dashing to cut the release from it.

piraka9011 and others added 7 commits January 28, 2020 10:29
Signed-off-by: Anas Abou Allaban <aabouallaban@pm.me>
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
Signed-off-by: Anas Abou Allaban <aabouallaban@pm.me>
Signed-off-by: Anas Abou Allaban <allabana@amazon.com>
* use std::is_pointer for macro
* linters
* use yaml_cpp_vendor
* add test_robot_hardware to metapackage
* use rcpputils filesystem helper
* export yaml-cpp symbols
@Karsten1987 Karsten1987 self-assigned this Feb 5, 2020
@Karsten1987
Copy link
Contributor Author

Karsten1987 commented Feb 5, 2020

CI including ros_controllers & ci script karsten_ros2_control for enabling trajectory_msgs:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status
  • Windows-container Build Status

@Karsten1987
Copy link
Contributor Author

Addressing Windows warnings in ros2_controllers:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status
  • Windows-container Build Status

@Karsten1987 Karsten1987 merged commit 45d71ac into dashing Feb 6, 2020
@Karsten1987 Karsten1987 deleted the dashing_update branch February 6, 2020 00:49
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.

3 participants