Skip to content

fix(add): add static folder to .prettierignore #618

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

Merged
merged 1 commit into from
Jun 28, 2025

Conversation

manuel3108
Copy link
Member

Closes #607

This is probably debatable, but based on the presentation inside the docs, this does make sense in my opinion. I was currently unable to imagine any valid use-case where you would want to format your static folder.

Copy link

changeset-bot bot commented Jun 28, 2025

🦋 Changeset detected

Latest commit: 13e6505

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

Copy link

pkg-pr-new bot commented Jun 28, 2025

Open in StackBlitz

npx https://pkg.pr.new/sveltejs/cli/sv@618
npx https://pkg.pr.new/sveltejs/cli/svelte-migrate@618

commit: 13e6505

@manuel3108 manuel3108 merged commit aed35a3 into main Jun 28, 2025
12 of 17 checks passed
@manuel3108 manuel3108 deleted the fix/static-in-prettierignore branch June 28, 2025 09:04
@github-actions github-actions bot mentioned this pull request Jun 28, 2025
@bensgilbert
Copy link

@manuel3108, the static/ pattern will ignore any directory named static anywhere in the project, alternatively the /static/ pattern will ignore just the static folder at the root of the project (the intended behaviour). This will avoid confusion when (for whatever reason), the programmer has a folder named static somewhere else in their project. Would you mind if I made a pr changing it to /static/?

@manuel3108
Copy link
Member Author

Ahhh shit, you are obviously right. Happy to merge a pr

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.

add /static to prettierignore file
2 participants