fix: ignore hidden directories in empty directory detection

[manuel3108]

Relates #114 (not closes)

This only ignores the hidden directory. This is a quick win we can get up and ready before summit. We can always re-iterate that thing later on.

[changeset-bot]

🦋 Changeset detected

Latest commit: c715409

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
sv	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/sveltejs/cli/sv@126

commit: c715409

[Conduitry]

Are there other hidden directories that we'd actually want to work this way, or is it really just .git? If there are other . directories or files present in a directory, who knows what's going on with it. I don't think it necessarily feels safe to just assume we're good to go without warning the user.

[benmccann]

Personally, I don't see a strong need to change anything here. I just tested and all it did was prompted me:

Directory not empty. Continue?

Perhaps that's because my workflow doesn't involve creating a new repo and then running sv create. I would do it in the other order, which I think is what GitHub encourages

But I'm not opposed to ignoring .git in this check if folks would like

[manuel3108]

Sure, we could just limit it to .git instead of ., as that's probably most used hidden directory on my computer.

Since this already came up in svelte-add and not many people knew about this tool at all, I'd assume this will come up here in a matter of days.