ros2 / ci

ROS 2 CI Infrastructure
http://ci.ros2.org/
Apache License 2.0
48 stars 30 forks source link

Remove the ability to use a venv. #751

Closed clalancette closed 4 months ago

clalancette commented 5 months ago

In all cases, we are now building with containers, which means venv are an unnecessary container-within-a-container.

This will fix #400 by side-stepping the problem completely.

This depends on #750 going in first, as we need the Dockerfiles to install colcon.

clalancette commented 5 months ago

CI:

clalancette commented 4 months ago

And I've now deployed this to all jobs.