ManageIQ / awesome_spawn

AwesomeSpawn is a module that provides some useful features over Ruby's Kernel.spawn.
MIT License
9 stars 16 forks source link

add pid to command result #49

Closed d-m-u closed 4 years ago

d-m-u commented 4 years ago

i'm fixing a brakeman warning in system_console about pid = spawn(*command) and i'd like to have command result be able to return the pid

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 2d9b0f2196ac58a110c175265cdc34ff23a11c62 on d-m-u:add_pid_to_command_result into ca555de95ab3dd5cc8cfe9724623b8fd48f647f5 on ManageIQ:master.

miq-bot commented 4 years ago

Checked commit https://github.com/d-m-u/awesome_spawn/commit/2d9b0f2196ac58a110c175265cdc34ff23a11c62 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint 5 files checked, 2 offenses detected

lib/awesome_spawn.rb

kbrock commented 4 years ago

Would be nice to merge #32 and this one. since that is from 2015, I think it is time to get it working, or just close it