Skip to content

Add sveltekit build and dist #4691

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add sveltekit build and dist #4691

wants to merge 1 commit into from

Conversation

cellulosa
Copy link

Reasons for making this change

This update excludes the dist/ and build/ directories from version control.

Why:

  • build/: This folder is generated when running vite build in SvelteKit apps. It’s a temporary output used for deployment and doesn’t need to be tracked in Git.
  • dist/: This is commonly used in Svelte libraries or packages as the output directory for compiled code. Like build/, it’s a generated artifact and should not be committed.

Tracking these folders can cause unnecessary noise in the repository, potential merge conflicts, and confusion over what’s source vs. what’s generated.

These directories should be reproducible from source using build commands (pnpm build, turbo build, etc.), so there’s no need to keep them in version control.

Links to documentation supporting these rule changes

Merge and Approval Steps

  • Confirm that you've read the contribution guidelines and ensured your PR aligns
  • Ensure CI is passing
  • Get a review and Approval from one of the maintainers

@cellulosa cellulosa requested a review from a team as a code owner July 26, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant