Add timeouts into the ExecutionModifiers

[fahhem]

This pipes run_proc's timeout argument farther up the stack.

I'm hoping the tests will catch if this hits any parts that don't accept timeout and I'll add it to those as well.

[coveralls]

Coverage increased (+0.03%) to 80.167% when pulling ef188ae on fahhem:patch-1 into e287d46 on tomerfiliba:master.

[coveralls]

Coverage increased (+0.03%) to 80.167% when pulling 0b9e482 on fahhem:patch-1 into e287d46 on tomerfiliba:master.

[fahhem]

Looks like the only failure in travis is due to Python nightly changing the string representation of ints. Should I fix that in this PR or a separate one?

[henryiii]

One PR is okay with me.
Henry

On Jun 10, 2016, at 11:20 AM, Fahrzin Hemmati notifications@github.com wrote:

Looks like the only failure in travis is due to Python nightly changing the string representation of ints. Should I fix that in this PR or a separate one?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #281 (comment), or mute the thread https://github.com/notifications/unsubscribe/AEZyyvHFv85MTqcRaanNSGJ31BGYRkrSks5qKSxKgaJpZM4IyeD-.

[coveralls]

Coverage increased (+0.004%) to 80.143% when pulling 0f505ad on fahhem:patch-1 into e287d46 on tomerfiliba:master.

[fahhem]

Okay, the latest push should have passing tests

[henryiii]

Okay, I’ll look at it when I get off work.
Thanks!
Henry

On Jun 10, 2016, at 12:16 PM, Fahrzin Hemmati notifications@github.com wrote:

Okay, the latest push should have passing tests

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub #281 (comment), or mute the thread https://github.com/notifications/unsubscribe/AEZyyrYViMFLtGbWnow9_31gLi4Frn6Iks5qKTlmgaJpZM4IyeD-.

[coveralls]

Coverage increased (+0.03%) to 80.167% when pulling 3499955 on fahhem:patch-1 into e287d46 on tomerfiliba:master.