Add TODOs and comments

Author: gleichdick

tf2/include/tf2/impl/convert.h

@@ -90,6 +90,8 @@ struct DefaultMessageForDatatype
     alwaysFalse<Datatype>,
     "No default message type defined, "
     "please check your header file includes!");
+
+  // implement the following in the specializations for your own types
   // using type = ...;
 };
 
@@ -115,6 +117,8 @@ struct DefaultTransformType
     alwaysFalse<Datatype>,
     "No default transform type defined, "
     "please check your header file includes!");
+
+  // implement the following in the specializations for your own types
   // using type = ...;
 };
 
@@ -146,6 +150,8 @@ struct ConversionImplementation
     alwaysFalse<Datatype>,
     "No Conversion Implementation available, "
     "please check your header file includes!");
+
+  // implement the following in the specializations for your own types
   // void toMsg(const Datatype&, Message&);
   // void fromMsg(const Message&, Datatype&);
 };
@@ -190,6 +196,8 @@ struct StampedMessageTraits
     alwaysFalse<StampedMessage>,
     "No traits for this stamped message type available, "
     "please check your header file includes!");
+
+  // implement the following in the specializations for your own types
   // using UntampedType = ...;
   // static UntampedType& accessMessage(StampedMsg &);
   // static UntampedType getMessage(StampedMsg const&);
@@ -226,6 +234,8 @@ struct UnstampedMessageTraits
     alwaysFalse<UnstampedMessage>,
     "No traits for this message type available, "
     "please check your header file includes!");
+
+  // implement the following in the specializations for your own types
   // using StampedType = ...;
   // using StampedTypeWithCovariance = ...;
 };

tf2_geometry_msgs/CMakeLists.txt

@@ -31,6 +31,8 @@ if(BUILD_TESTING)
   find_package(ament_cmake_gtest REQUIRED)
   find_package(rclcpp REQUIRED)
 
+  # TODO(ros2/geometry2#259) Switch to ament_lint_auto once headers
+  # are renamed to .hpp
   find_package(ament_cmake_cppcheck REQUIRED)
   find_package(ament_cmake_cpplint REQUIRED)
   find_package(ament_cmake_lint_cmake REQUIRED)

tf2_kdl/CMakeLists.txt

@@ -34,6 +34,8 @@ if(BUILD_TESTING)
   find_package(rclcpp REQUIRED)
   find_package(tf2_msgs REQUIRED)
 
+  # TODO(ros2/geometry2#259) Switch to ament_lint_auto once headers
+  # are renamed to .hpp
   find_package(ament_cmake_cppcheck REQUIRED)
   find_package(ament_cmake_cpplint REQUIRED)
   find_package(ament_cmake_lint_cmake REQUIRED)

tf2_sensor_msgs/CMakeLists.txt

@@ -30,6 +30,7 @@ if(BUILD_TESTING)
   find_package(ament_cmake_gtest REQUIRED)
   find_package(ament_cmake_pytest REQUIRED)
 
+  # TODO(ros2/orocos_kinematics_dynamics): reenable when PyKDL is ready to use
   #ament_add_pytest_test(test_tf2_sensor_msgs_py test/test_tf2_sensor_msgs.py)
 
   ament_add_gtest(test_tf2_sensor_msgs_cpp test/test_tf2_sensor_msgs.cpp)