-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Separate interface package system_modes_msgs #74
Conversation
Separated msg and srv types into seperate package . This seems to be the ROS 2 way anyway, was requiring a hack beforehand, and seems necessary to get windows build working. Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Adapts system_modes and system_modes_example packages to separated system_modes_msgs interface package. Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good work. One finding only in system_modes/package.xml.
Signed-off-by: Nordmann Arne (CR/ADT3) <arne.nordmann@de.bosch.com>
Codecov Report
@@ Coverage Diff @@
## master #74 +/- ##
==========================================
- Coverage 32.63% 31.87% -0.77%
==========================================
Files 14 14
Lines 1048 1048
Branches 754 754
==========================================
- Hits 342 334 -8
- Misses 132 136 +4
- Partials 574 578 +4
Continue to review full report at Codecov.
|
Separated msg and srv types into seperate package
system_modes_msgs
. This seems to be the ROS 2 way anyway, was requiring a hack beforehand, and seems necessary to get the windows build working. (I have a working windows build based on this change now)