89f291c007
Closes #15250 This PR simplifies our Vite integration even more. It turns out that in some projects (see #15250 for the exact repro), the way we invoke `svelte-preprocess` was actually causing issues in Vite since with Vite, it's expected to use the `sveltePreprocess` version exported by `sveltejs/vite-plugin-svelte`. While trying to change this we noticed that there are different versions of `sveltejs/vite-plugin-svelte` for Vite 5 and Vite 6 which caused us to investigate even more and we noticed that we do not even need to recursively call into the `sveltePreprocess()` as every plugin is run after each other anyways. This allows us to drop the dependency on `svelte-preprocess` and simplify the code a bit more, registering only a `(string) => string` style transformer. ## Test Plan This was tsted on the repro repo from #15250 as well as the SvelteKit setup from [my playgrounds](https://github.com/philipp-spiess/tailwindcss-playgrounds). Furthermore we tested various combinations of `svelte`, `@sveltejs/vite-plugin-svelte` and `vite` in our integration test to ensure everything works as expected. --------- Co-authored-by: Jordan Pittman <jordan@cryptica.me> |
||
|---|---|---|
| .. | ||
| astro.test.ts | ||
| config.test.ts | ||
| css-modules.test.ts | ||
| index.test.ts | ||
| multi-root.test.ts | ||
| nuxt.test.ts | ||
| other-transforms.test.ts | ||
| resolvers.test.ts | ||
| svelte.test.ts | ||
| url-rewriting.test.ts | ||
| vue.test.ts | ||