Skip to content
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.

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

Better fail clone management #572

Merged
merged 7 commits into from Apr 23, 2014

Conversation

pmiossec
Copy link
Member

Don't cleanup clone directory when fail fetch during clone

It's a shame to end with a clean directory after (perhaps)
hours of fetching history because a problem during a fetch

@spraints spraints mentioned this pull request Mar 16, 2014
12 tasks
It's a shame to end with a clean directory after (perhaps)
hours of fetching history because a problem during a fetch
@KindDragon
Copy link
Contributor

Than you can remove --resumable Clone command line option. User can continue cloning with fetch or branch --init -all

@spraints
Copy link
Member

🤘

Could git-tfs write a message that you can cd Project and git tfs fetch --all (or whatever) to finish the clone?

@spraints
Copy link
Member

Could git-tfs write a message that you can cd Project and git tfs fetch --all (or whatever) to finish the clone?

... can be something we do in the future.

This is the last thing on the "must have" list for v0.20 (#559). Is it still good to merge? If it is, @pmiossec feel free to hit the Merge button, and I can cut a release this week.

@pmiossec
Copy link
Member Author

@spraints I have displayed the message :) Some refactoring, correction and improvements was needed. Not we could merge...

spraints added a commit that referenced this pull request Apr 23, 2014
@spraints spraints merged commit 39a29e3 into git-tfs:master Apr 23, 2014
@pmiossec pmiossec deleted the better_fail_clone_management branch April 28, 2014 16:54
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.

None yet

3 participants