Skip to content

Move cd() call to the low-level wrap-call() function#142

Merged
sbesson merged 2 commits intoome:masterfrom
sbesson:118_chdir_decorator
Feb 6, 2014
Merged

Move cd() call to the low-level wrap-call() function#142
sbesson merged 2 commits intoome:masterfrom
sbesson:118_chdir_decorator

Conversation

@sbesson
Copy link
Copy Markdown
Member

@sbesson sbesson commented Feb 5, 2014

This commit implies any GitRepository method using a low-level git ...
command does not need to care about moving to the right location.

Closes #118

This commit implies any GitRepository method using a low-level `git ...`
command does not need to care about moving to the right location.
@joshmoore
Copy link
Copy Markdown
Member

If all tests seem happy, makes sense: http://ci.openmicroscopy.org/view/Mgmt/job/SCC-self-merge/240/console

@manics
Copy link
Copy Markdown
Member

manics commented Feb 6, 2014

Looks good to merge

sbesson added a commit that referenced this pull request Feb 6, 2014
Move cd() call to the low-level wrap-call() function
@sbesson sbesson merged commit e208ef4 into ome:master Feb 6, 2014
@sbesson sbesson deleted the 118_chdir_decorator branch February 6, 2014 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove all calls to os.chdir

3 participants