search

ros2 / rcpputils

Apache License 2.0
32 stars 55 forks source link

Included tl_expected #185

Closed ahcorde closed 7 months ago

ahcorde commented 7 months ago

This PR is related with this other PR https://github.com/ros2/ros2/pull/1516

The idea is to include tl_expected in rclcpp until the standard CXX20 is available in the ros distribution

FYI @tylerjw

ahcorde commented 7 months ago
ahcorde commented 7 months ago

@clalancette can we also backport this to humble and iron ?

ahcorde commented 7 months ago

https://github.com/Mergifyio backport humble iron

mergify[bot] commented 7 months ago

backport humble iron

✅ Backports have been created

* [#186 Included tl_expected (backport #185)](https://github.com/ros2/rcpputils/pull/186) has been created for branch `humble` but encountered conflicts * [#187 Included tl_expected (backport #185)](https://github.com/ros2/rcpputils/pull/187) has been created for branch `iron`