Fix example of Job Queue #1313

Merged
merged 2 commits into from Apr 14, 2015

Projects

None yet

2 participants

@crlane
Contributor
crlane commented Apr 13, 2015

Fixes an example for the job queue constructor (which now requires a second positional argument, intended to be a Queue() object)

Also addresses #1305 (comment) . We'll now only set exit status if it's a process and do nothing if it's a thread.

crlane added some commits Apr 13, 2015
@crlane crlane [#1305] Add queue for try_using examples f6cc61b
@crlane crlane [#1305] Only alter the exit_code if job is a Process
- Addresses this comment on issue:
   fabric#1305 (comment)
- no status code to set for Threads anyway
- only processes used in fabric internals
f97c9db
@crlane crlane changed the title from Cl/jq docs to Fix example of Job Queue Apr 13, 2015
@bitprophet bitprophet added a commit that referenced this pull request Apr 14, 2015
@bitprophet bitprophet Changelog, fixes #1305, fixes #1313 0140537
@bitprophet bitprophet merged commit f97c9db into fabric:1.9 Apr 14, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment