Alberto Soragna
Alberto Soragna
Hi, can we get a review here?
Hi, anyone available for a review?
Friendly ping for a review
@wjwwood @clalancette @audrow @hidmic @ivanpauno can we get a review? this PR has been open for more than 1 year
Hi @DensoADAS, this is a 1-year old PR and it looks like it has been completely rebased and/or reworked. It will require a full review from scratch before proceeding. I'll...
Sorry for the wait. CI build: * Linux [](http://ci.ros2.org/job/ci_linux/19845/) * Linux-aarch64 [](http://ci.ros2.org/job/ci_linux-aarch64/14343/) * Windows [](http://ci.ros2.org/job/ci_windows/20566/)
The PR looks good. If possible, I would ask to add a few more unit-tests to validate the changes in the intra-process manager
Hi, any update on this?
CI with repos https://gist.github.com/alsora/2cb20dc394c9722625b409f4a8566c5f * Linux [](http://ci.ros2.org/job/ci_linux/18667/) * Linux-aarch64 [](http://ci.ros2.org/job/ci_linux-aarch64/13195/) * Windows [](http://ci.ros2.org/job/ci_windows/19396/)
Running again as the failure seems unrelated. * Linux [](http://ci.ros2.org/job/ci_linux/18685/) * Linux-aarch64 [](http://ci.ros2.org/job/ci_linux-aarch64/13211/) * Windows [](http://ci.ros2.org/job/ci_windows/19413/)