-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
create-svelte
output is missing .gitignore
#1036
Comments
Should we first ask people whether they want to use Git? (And default to yes.) And if they do, then not only create .gitignore, but also run git init for them? Even if they don't plan on pushing anywhere yet, they can at least see in their editor which files are generated/ignored if the editor grays them out or whatever. |
I'm not sure I'd want |
It should be pretty much a no-op if you run |
Yeah, looks like basically a no-op. I just tested and it says "Reinitialized existing Git repository" |
I'd vote no on However, I think adding too many questions to the template is something to avoid. Instead of asking "Do you want to use Git?" and then running |
I agree with too many steps being a concern but this step feels important enough for me. It's also the default for |
For anyone that runs into this issue, you can solve it by copying this |
Nuxt also does it that way:
Personally, I find the 'select option' method to be a smoother experience than being suggested code to copy & paste. |
I PR'd this issue without actually reading the thread above — will create a separate issue about |
Describe the bug
create-svelte
output is missing.gitignore
To Reproduce
Expected behavior
.gitignore
is included in the default templateSeverity
Oops.
Additional context
Probably got missed by accident when we re-wrote the template to generate the JS version from the single TS version.
The text was updated successfully, but these errors were encountered: