search

sveltejs / cli

The Svelte CLI
MIT License
258 stars 11 forks source link

'next steps' should omit `git init` if we are already in a git repo #287

Open Rich-Harris opened 1 week ago

Rich-Harris commented 1 week ago

If you're creating a new app inside a git repo, 'next steps' includes this...

git init && git add -A && git commit -m "Initial commit" (optional)

...and it should probably just say nothing

dominikg commented 1 week ago

i'd be in favor of turning the option around entirely automatically call git init unless --no-git-init was passed or we are in a git project already. if one was initialized, add "we initialized git for you, if you don't need that, run `rm -rf .git" to next steps

(edit: only if git is installed, we are in a tty and CI is false)

AdrianGonz97 commented 1 week ago

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