methylDragon comments

Results 155 comments of


                                            methylDragon

Kill dangling subprocesses

@ivanpauno just some flake8 linting issues

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17842)](http://ci.ros2.org/job/ci_linux/17842/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12410)](http://ci.ros2.org/job/ci_linux-aarch64/12410/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18510)](http://ci.ros2.org/job/ci_windows/18510/)

Allow ReadyToTest() usage in event handler

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17734)](http://ci.ros2.org/job/ci_linux/17734/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12275)](http://ci.ros2.org/job/ci_linux-aarch64/12275/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18377)](http://ci.ros2.org/job/ci_windows/18377/)

Allow ReadyToTest() usage in event handler

Thank you for the contribution! :fire:

Pass modules to PythonExpression

Could you add tests? @RobertBlakeAnderson

Pass modules to PythonExpression

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17828)](http://ci.ros2.org/job/ci_linux/17828/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12395)](http://ci.ros2.org/job/ci_linux-aarch64/12395/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18497)](http://ci.ros2.org/job/ci_windows/18497/)

Pass modules to PythonExpression

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17848)](http://ci.ros2.org/job/ci_linux/17848/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12416)](http://ci.ros2.org/job/ci_linux-aarch64/12416/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18516)](http://ci.ros2.org/job/ci_windows/18516/) :fire:

Pass modules to PythonExpression

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17850)](http://ci.ros2.org/job/ci_linux/17850/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12418)](http://ci.ros2.org/job/ci_linux-aarch64/12418/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18518)](http://ci.ros2.org/job/ci_windows/18518/)

Pass modules to PythonExpression

Flake 8 issues :o https://ci.ros2.org/job/ci_linux-aarch64/12418/testReport/launch.test/test_flake8/test_flake8/ I think just change the name `locals`

Pass modules to PythonExpression

* Linux [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux&build=17854)](http://ci.ros2.org/job/ci_linux/17854/) * Linux-aarch64 [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_linux-aarch64&build=12422)](http://ci.ros2.org/job/ci_linux-aarch64/12422/) * Windows [![Build Status](http://ci.ros2.org/buildStatus/icon?job=ci_windows&build=18523)](http://ci.ros2.org/job/ci_windows/18523/)