ros-infrastructure / ros_buildfarm

ROS buildfarm based on Docker
Apache License 2.0
81 stars 96 forks source link

Declare test dependencies in [test] extra #967

Closed cottsay closed 2 years ago

cottsay commented 2 years ago

Formally declaring the test dependencies somewhere is a good idea, and allows tools like colcon to discover them.

['extras_require']['test'] is where we put these dependencies in colcon itself, but there doesn't appear to be a strong consensus among the packaging community regarding where to list these, and many options aren't compatible with python 2, which is still supported in this package. This option seems to meet all of our needs.