search

ros2 / rosidl_dds

Apache License 2.0
6 stars 7 forks source link

Expose .idl to DDS .idl conversion via rosidl translate CLI #55

Closed hidmic closed 3 years ago

hidmic commented 3 years ago

Connected to https://github.com/ros2/rosidl/issues/565. Depends on https://github.com/ros2/rosidl/pull/575.

ahcorde commented 3 years ago

The CI job needs a release https://build.ros2.org/job/Rpr__rosidl_dds__ubuntu_focal_amd64/5/ for rosidl_cli, right ?

ahcorde commented 3 years ago

@osrf-jenkins retest this please

hidmic commented 3 years ago

The CI job needs a release https://build.ros2.org/job/Rpr__rosidl_dds__ubuntu_focal_amd64/5/ for rosidl_cli, right ?

Indeed!

Edit: https://github.com/ros/rosdistro/pull/28525 is up

ahcorde commented 3 years ago

@osrf-jenkins retest this please

hidmic commented 3 years ago

Running CI up to rosidl_generator_dds_idl:

hidmic commented 3 years ago

Re-running Windows CI after https://github.com/ros2/rosidl_dds/pull/55/commits/4bbea01e9d221b2b79cd55642f1fa32426b37795:

hidmic commented 3 years ago

Alright, CI's green, reviewer is OK with this patch. Going in!