ros2 / ci

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

Pin the versions of flake8. #675

Closed clalancette closed 2 years ago

clalancette commented 2 years ago

This also caused us to refactor how we are doing the discovery of pip packages. The new version is actually shorter.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

clalancette commented 2 years ago

Here's some starting CI, just up to rcutils, to make sure things are working as expected. We'll greatly expand what we test from there:

clalancette commented 2 years ago

OK, here is a similar test on Foxy:

clalancette commented 2 years ago

I've had to do some things here that I don't love, but at least I'm getting green results on both Foxy and Rolling. Here's an expanded set of jobs on both Foxy and Rolling to see what happens:

Rolling (all packages):

Foxy (all packages):

clalancette commented 2 years ago

Here's another set of complete runs, which should hopefully fix the RHEL issue:

Rolling (all packages):

Foxy (all packages):

clalancette commented 2 years ago

This did not end up working well at all, as you can see by all of the yellow and red above. Closing this in favor of #677 , which I think will work better.