ros2 / launch

Tools for launching multiple processes and for writing tests involving multiple processes.
Apache License 2.0
124 stars 139 forks source link

Fix flake8 errors. #695

Closed clalancette closed 1 year ago

clalancette commented 1 year ago

Introduced by #686 . @methylDragon FYI.

clalancette commented 1 year ago

CI:

methylDragon commented 1 year ago

o__o I'm not sure why CI didn't catch that

clalancette commented 1 year ago

o__o I'm not sure why CI didn't catch that

At the time you ran the CI for this, there was a bug in colcon that failed to run any tests. So in point of fact, none of the tests ran for this change. That bug has since been fixed, but this is the fallout from that.