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

make init-branch command obsolete! #452

Merged
merged 2 commits into from Oct 1, 2013

Conversation

pmiossec
Copy link
Member

and correct some indentation problems...

and correct some indentation problems...
@pmiossec pmiossec mentioned this pull request Sep 30, 2013
@sc68cal
Copy link
Contributor

sc68cal commented Sep 30, 2013

👍 for this - so that we can eventually merge #393

@sc68cal
Copy link
Contributor

sc68cal commented Sep 30, 2013

@pmiossec - how about a similar commit to tee up #450 ?

@spraints
Copy link
Member

spraints commented Oct 1, 2013

🆒 Do you think there should be a warning when the command gets run, too?

@pmiossec
Copy link
Member Author

pmiossec commented Oct 1, 2013

@spraints that what I did in the first time but I removed it because that's exactly the same come that is run with command branch --init and it will be displayed even when the good command is called :(

I could add some dirty code to manage this case (add a property set to not display the message). That should not be a big problem because this code will be temporary and removed when the init-branch command will be definitely removed. What do you think of that?

@pmiossec
Copy link
Member Author

pmiossec commented Oct 1, 2013

i did it...

spraints added a commit that referenced this pull request Oct 1, 2013
@spraints spraints merged commit 2802474 into git-tfs:master Oct 1, 2013
@pmiossec pmiossec deleted the obsolete_init_branch_command branch October 1, 2013 15:01
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