ros2 / rosidl_typesupport

Packages which provide the typesupport for ROS messages and services
Apache License 2.0
13 stars 34 forks source link

typesupport_tests needs to be updated to C++17 #137

Closed CursedRock17 closed 1 year ago

CursedRock17 commented 1 year ago

Similar to PR #131 most of ros 2 support should be pushed towards C++17 and rosidl_typesupport_tests should be included to be consistent. Also, this was causing errors on MacOS and if it's such a miniscule change that doesn't cause issues in other places, there is no reason not to add C++17 support.

clalancette commented 1 year ago

CI: