Skip to content

Conversation

eltigerchino
Copy link
Member

Accompanying sveltejs/kit#14026

This PR changes the comment in the create template's tsconfig.json file to prefer the kit config setting for extending the tsconfig file instead of overriding the includes and excludes settings.

Open to a better explanation/wording of why the setting should be used.

Copy link

pkg-pr-new bot commented Aug 14, 2025

Open in StackBlitz

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

commit: 12cd03c

Copy link

changeset-bot bot commented Aug 14, 2025

🦋 Changeset detected

Latest commit: 12cd03c

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

@manuel3108 manuel3108 changed the title feat: recommend the kit.typescript.config setting instead of copying from the generated config chore: recommend the kit.typescript.config setting instead of copying from the generated config Aug 14, 2025
@manuel3108 manuel3108 changed the title chore: recommend the kit.typescript.config setting instead of copying from the generated config chore(creeate): recommend the kit.typescript.config setting instead of copying from the generated config Aug 14, 2025
@manuel3108 manuel3108 changed the title chore(creeate): recommend the kit.typescript.config setting instead of copying from the generated config chore(create): recommend the kit.typescript.config setting instead of copying from the generated config Aug 14, 2025
@manuel3108 manuel3108 merged commit 01fe82a into main Aug 14, 2025
7 checks passed
@manuel3108 manuel3108 deleted the eltigerchino-patch-1 branch August 14, 2025 17:17
@github-actions github-actions bot mentioned this pull request Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants