Queued spawn

[bmeck]

fixes #5

this uses process-queue in order to allow for some other business requirements I had such as being able to change the spawning function (for remote spawning), and logging the processes.

[diosney]

Nice! Thanks! 😄

Later, I will change some code styling on var names to maintain the same overall code style, I hope you don't mind :)

[bmeck]

changing it is fine, might also want to change the undocumented cmdQueue
option to something that fits your style.

On Fri, Sep 19, 2014 at 9:35 AM, Diosney Sarmiento <[email protected]

wrote:

Merged #6 #6.

—
Reply to this email directly or view it on GitHub
#6 (comment).

[diosney]

Great!

Also, there was a dependency that was giving errors on install, namely offspring, I removed it from package.json and everything was smoothly.

[diosney]

@bmeck: I finished the revision, added several basic tests (which was due long ago), and released a new version with the changes.

If you find anything is missing or wrong just tell.

BTW: I changed cmdQueue to only be queue and deprecated the specific .comment in favor of the generic matches approach, hope you don't mind.

And again, thanks for your help, it is more than welcomed :D

[bmeck]

seems fine. +1

On Sun, Sep 28, 2014 at 8:34 PM, Diosney Sarmiento <[email protected]

wrote:

@bmeck https://github.com/bmeck: I finished the revision, added several
basic tests (which was due long ago), and released a new version with the
changes.

If you find anything is missing or wrong just tell.

BTW: I changed cmdQueue to only be queue and deprecated the specific
.comment in favor of the generic matches approach, hope you don't mind.

And again, thanks for your help, it is more than welcomed :D

—
Reply to this email directly or view it on GitHub
#6 (comment).