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

chore: overhaul ignore configs #11938

Merged
merged 12 commits into from
May 30, 2024
5 changes: 5 additions & 0 deletions .changeset/ten-wolves-hug.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"create-svelte": patch
---

chore: overhaul ignore configs
15 changes: 7 additions & 8 deletions packages/create-svelte/shared/+eslint/.eslintignore
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
.DS_Store
/package
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wouldn't this be considered "Output"?

I think if you changed the "Lockfiles" head to "Package managers" then you could put node_modules there too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't certain about the /packages directory. When is it used? The package template uses the dist directory.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps we don't need it anymore. it may be leftover. I think svelte-kit package used to output to that location

node_modules
/build

# Output
/.svelte-kit
/package
.env
.env.*
!.env.example
/build
/dist

# Ignore files for PNPM, NPM and YARN
hyunbinseo marked this conversation as resolved.
Show resolved Hide resolved
pnpm-lock.yaml
# Lockfiles
package-lock.json
pnpm-lock.yaml
yarn.lock
4 changes: 2 additions & 2 deletions packages/create-svelte/shared/+prettier/.prettierignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Ignore files for PNPM, NPM and YARN
pnpm-lock.yaml
# Lockfiles
package-lock.json
pnpm-lock.yaml
yarn.lock
21 changes: 15 additions & 6 deletions packages/create-svelte/templates/default/.gitignore
Original file line number Diff line number Diff line change
@@ -1,12 +1,21 @@
.DS_Store
.output
.vercel
/package
node_modules
/build

# Output
hyunbinseo marked this conversation as resolved.
Show resolved Hide resolved
/.svelte-kit
/package
.env
hyunbinseo marked this conversation as resolved.
Show resolved Hide resolved
/build

# OS
.DS_Store
Thumbs.db

# Env
.env.*
!.env.example
.vercel
.output
!.env.test

# Vite
vite.config.js.timestamp-*
vite.config.ts.timestamp-*
17 changes: 13 additions & 4 deletions packages/create-svelte/templates/skeleton/.gitignore
Original file line number Diff line number Diff line change
@@ -1,10 +1,19 @@
.DS_Store
/package
node_modules
/build

# Output
/.svelte-kit
/package
.env
/build

# OS
.DS_Store
Thumbs.db

# Env
.env.*
!.env.example
!.env.test

# Vite
vite.config.js.timestamp-*
vite.config.ts.timestamp-*
17 changes: 13 additions & 4 deletions packages/create-svelte/templates/skeletonlib/.gitignore
Original file line number Diff line number Diff line change
@@ -1,11 +1,20 @@
.DS_Store
/package
node_modules

# Output
/.svelte-kit
/build
/dist
/.svelte-kit
/package
.env

# OS
.DS_Store
Thumbs.db

# Env
.env.*
!.env.example
!.env.test

# Vite
vite.config.js.timestamp-*
vite.config.ts.timestamp-*