search

ros-controls / ros2_control_ci

This repository holds reusable workflows for CI of the ros2_control framework.
https://control.ros.org
Apache License 2.0
2 stars 1 forks source link

Adds coverage.py #115

Closed christophfroehlich closed 1 month ago

christophfroehlich commented 1 month ago

This is necessary to get branch coverage of python files called within c++ tests.

christophfroehlich commented 1 month ago

@fmauch I had to fight the venv settings, it seems that action-ros-ci cleans the workspace and a venv created in .venv does not exist afterwards (this is why I create it now in the user folder ~/.venv).

and still the codecov-action does not find the coverage.py and throws this warning

https://github.com/codecov/codecov-cli/blob/1478514b1203d1c3921ca60b767c4224d0c417e4/codecov_cli/plugins/pycoverage.py#L58-L60

Maybe you see an obvious issue with these changes or have other suggestions..