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

sprout from current branch if only 1 argument is provided #37

Closed
franciscolourenco opened this issue Mar 25, 2012 · 12 comments
Closed

sprout from current branch if only 1 argument is provided #37

franciscolourenco opened this issue Mar 25, 2012 · 12 comments

Comments

@franciscolourenco
Copy link

I tried to "git sprout test" and expected a test branch to be created from the current branch, but instead it complained there was no branch called test.

when providing only one argument, sprout should branch from the current branch into a new branch with the name passed in the argument

@kennethreitz
Copy link
Collaborator

Absolutely, thanks!

@cayblood
Copy link

What is the use case for a multi-arg version of sprout? The one in the docs is single arg.

@franciscolourenco
Copy link
Author

it seems some of docs are not consistently up to date since they are spread over at least 3 locations

On Tuesday, March 27, 2012 at 11:57 , Carl Youngblood wrote:

What is the use case for a multi-arg version of sprout? The one in the docs is single arg.


Reply to this email directly or view it on GitHub:
#37 (comment)

@kennethreitz
Copy link
Collaborator

Readme is more specific than the website because I'd like the website to be as concise as possible.

Inconsistencies == bugs in the code.

@franciscolourenco
Copy link
Author

I understand.
I was referring to this particular page ( http://www.git-legit.org/ ) where many commands feature only 1 argument, but in reality they need two.

In the other 2 references I found ( http://kennethreitz.com/legit-the-sexy-git-cli.html , https://github.com/kennethreitz/legit ) the commands seem to better represent the current state.

It was in this sense that I meant there is an inconsistency between the first page and the 2 others as well with the current version. I thought this page was the main (It looks pretty good btw), and therefore don't understand why is out of date, or is it predicting future changes?

cheers :)

On Saturday, March 31, 2012 at 06:20 , Kenneth Reitz wrote:

Readme is more specific than the website because I'd like the website to be as concise as possible.

Inconsistencies == bugs in the code.


Reply to this email directly or view it on GitHub:
#37 (comment)

@kennethreitz
Copy link
Collaborator

@aristidesfl they should all require only one. the second parameter is supplementary.

If some of them don't work with one currently, that's a bug.

@franciscolourenco
Copy link
Author

that's great :)

On Saturday, March 31, 2012 at 17:07 , Kenneth Reitz wrote:

@aristidesfl they should all require only one. the second parameter is supplementary.

If some of them don't work with one currently, that's a bug.


Reply to this email directly or view it on GitHub:
#37 (comment)

@miguelbaldi
Copy link
Contributor

Seems to be working now, on 'develop' branch.

@franciscolourenco
Copy link
Author

There is currently no quick way to merge the current branch into another branch, neither with git nor with legit.

Shouldn't harvest take care of this when used with only one parameter?
Otherwise it is similar to git merge (when used only with one parameter)

@patcon
Copy link

patcon commented Sep 14, 2012

^^ @aristidesfl got me thinking :)

@borisrorsvort
Copy link

+1

@weakish
Copy link
Collaborator

weakish commented Jul 19, 2017

Close this since the sprout command has been removed.

@weakish weakish closed this as completed Jul 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants