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
chore: use peer dependencies rather than dev dependencies #11433
Conversation
🦋 Changeset detectedLatest commit: 39e4fe5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
This PR makes use of
|
Yarn PnP is broken, and we do not support it. Use npm or pnpm instead. |
Yarn PnP with Yarn 4 works on SvelteKit 2.0.7, but not on SvelteKit 2.0.8. |
Right now it's rather difficult to use Svelte 5 alongside a linked
@sveltejs/kit
, because Kit importsvite
and@sveltejs/vite-plugin-svelte
relative to itself, even though these are peer dependencies.This means that components will be compiled by Svelte 4 (as that's what the
@sveltejs/vite-plugin-svelte
inside the linked@sveltejs/kit
depends on), but then try to use the Svelte 5 runtime.This is already a nuisance but it will become much more of one as Svelte 5 gets closer to release and we're trying to make sure everything works in both versions.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm test
and lint the project withpnpm lint
andpnpm check
Changesets
pnpm changeset
and following the prompts. Changesets that add features should beminor
and those that fix bugs should bepatch
. Please prefix changeset messages withfeat:
,fix:
, orchore:
.