Skip to content

feat(create-vite): align tsconfigs in svelte-ts template with others#18995

Merged
patak-cat merged 1 commit into
vitejs:mainfrom
sapphi-red:feat/create-vite-svelte-ts-tsconfig
Dec 18, 2024
Merged

feat(create-vite): align tsconfigs in svelte-ts template with others#18995
patak-cat merged 1 commit into
vitejs:mainfrom
sapphi-red:feat/create-vite-svelte-ts-tsconfig

Conversation

@sapphi-red
Copy link
Copy Markdown
Member

Description

This PR aligns tsconfigs in svelte-ts template with other templates. This was not possible because an error was shown in the tsconfig in previous versions of the Svelte VSCode extension. But since v109.4.0, solution style tsconfigs does not show an error (sveltejs/language-tools@89c001b) any more.

fixes #18139 (indirectly)

@sapphi-red sapphi-red added p2-nice-to-have Not breaking anything but nice to have (priority) feat: create-vite create-vite package labels Dec 18, 2024
@dominikg
Copy link
Copy Markdown
Contributor

this is an improvement over the current status so 👍 but the overall setup is too complex for a simple svelte+ts.

It should not require a solution setup.

@patak-cat patak-cat merged commit b48b98c into vitejs:main Dec 18, 2024
@sapphi-red sapphi-red deleted the feat/create-vite-svelte-ts-tsconfig branch December 19, 2024 01:42
@brendanmatkin
Copy link
Copy Markdown

brendanmatkin commented Jan 7, 2025

@sapphi-red thanks!

edit: I manually applied these changes to an existing project, and it works well (note to others: make sure to restart your TS server or IDE if you get a "requires composite:true" warning)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: create-vite create-vite package p2-nice-to-have Not breaking anything but nice to have (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Referenced Project May Not Disable Emit

4 participants