-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
'next steps' should omit git init
if we are already in a git repo
#287
Comments
i'd be in favor of turning the option around entirely (edit: only if |
I think we should just get rid of it. It's unnecessary clutter for people that already know git, and it's a line full of funny words and symbols for those that don't. It's not very helpful in either case |
I think there's a bunch of different ways we could go with this. E.g. it could be:
We could also have a file like Then I think the other big question might be something like:
We could also potentially do nothing by default and not have any questions, but have text at the bottom of the screen that says something like |
Are
I think such a question would make sense. I'm not a fan of the preferences though, at least at the current stage. I do think this might make more sense in the future, once we integrate the other tools more deeply. |
I think that's about right
Which tools were you thinking of? |
So why bother about the other vcs? As we have done a number of times for other decision, we shouldn't bother about those minor minorities.
I was especially thinking about But as soon as we implement #278 i.e. you might be using |
Yeah, fair enough to not ask about the other version control software. Maybe we can combine the two questions then into a single question:
|
I'm still against adding a git option entirely, but if I had to choose, simply initializing the repo like this: #287 (comment) should be sufficient.
I don't think a |
I think asking if you want to initialize a git repo isn't very valuable if that's all you do. |
If you're creating a new app inside a git repo, 'next steps' includes this...
...and it should probably just say nothing
The text was updated successfully, but these errors were encountered: