Skip to content

add files with a separate add call in backport_pr#4141

Merged
takluyver merged 2 commits into
ipython:masterfrom
minrk:paul-was-wrong
Aug 29, 2013
Merged

add files with a separate add call in backport_pr#4141
takluyver merged 2 commits into
ipython:masterfrom
minrk:paul-was-wrong

Conversation

@minrk
Copy link
Copy Markdown
Member

@minrk minrk commented Aug 29, 2013

git commit [filename] will fail if the filename is not already tracked.

/cc @ivanov

`git commit [filename]` will fail if the filename is not already tracked.

/cc @ivanov
Comment thread tools/backport_pr.py Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can use call(), which returns the return code, rather than creating a Popen and wait()ing on it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup, I am using check_call elsewhere in the function, and should be using it here as well. Just an artifact of copy/pasting the wrong block.

@takluyver
Copy link
Copy Markdown
Member

Great, merging.

takluyver added a commit that referenced this pull request Aug 29, 2013
add files with a separate `add` call in backport_pr
@takluyver takluyver merged commit af884b0 into ipython:master Aug 29, 2013
minrk added a commit that referenced this pull request Sep 4, 2013
`git commit [filename]` will fail if the filename is not already tracked.

/cc @ivanov
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
add files with a separate `add` call in backport_pr
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.

2 participants