New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unify jobdirs #18

Merged

kofalt merged 2 commits into master from unify-jobdirs

Mar 26, 2015

Member

kofalt commented Mar 26, 2015

Adds a Configure(workspacePath string) method to executor interface
Added some quick documentation to a few files explaining expectations
Unify executor's usage of temp job directory logic

kofalt added 2 commits

March 25, 2015 21:32


          Attempt to unify temp-job directory logic and add some docs

461b9d8

Signed-off-by: Nathaniel Kofalt <nathaniel@kofalt.com>


          Rename function

b5cb905

Signed-off-by: Nathaniel Kofalt <nathaniel@kofalt.com>

Member

warpfork commented Mar 26, 2015

LGTM

kofalt added a commit that referenced this pull request


          Merge pull request #18 from polydawn/unify-jobdirs

0f8ebd7

Unify jobdirs

kofalt merged commit 0f8ebd7 into master

kofalt deleted the unify-jobdirs branch

March 26, 2015 03:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment