-
Notifications
You must be signed in to change notification settings - Fork 25
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
ROS2 support #32
Comments
My understanding is that is unnecessary (for at least C++) in ROS2 - since colcon natively supports code coverage: https://answers.ros.org/question/355388/generating-a-c-code-coverage-report-with-colcon/ - however I haven't actually tried this out yet. |
I tried it. I still need to create a baseline, add necessary flags, extract the right information from the report, generate html and so on. Is this not what makes your package necessary, or did I miss s.th. important? |
If anyone is looking for an example on how to do this for their repos in CI for ROS2 you can look at the moveit2 repo's github action workflow and our
If you wanted to do the same thing locally you could follow these steps:
I've used this workflow when writing tests and searching for the last few lines I don't have coverage of. |
Concerning CI coverage jobs, this might also be interesting: ros-industrial/industrial_ci#504 |
I just wanted to ask if you already had any thoughts or even progress in providing code-coverage for ros2 packages?
The text was updated successfully, but these errors were encountered: