Conversation
|
This is awesome, thank you so much! If you don't mind, I'll split this PR into separate branches so each changeset is self contained. For example, I'd rather wait to merge the QoS stuff until it's integrated with JNI, but I can't do it if everything is part of the same PR. BTW, I've just added you to https://github.com/esteve/ros2_java/blob/master/THANKS |
|
I've created the following branches based on this PR: https://github.com/esteve/ros2_java/tree/Theosakamg-master I haven't created separate branches for the library loader and the logger because both are too intertwined, please separate the functionalities so I we can discuss them independently. |
|
@Theosakamg I'm closing this pull request. It hasn't been updated and I haven't heard from you since I created the separate branches. Anyway, I have fully implemented QoS in master (see esteve@c2bd039) and added support for logging via SLF4J, which works much better on Android than If there's anything else you'd like to see in Thanks for your contribution! |
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Added :