Skip to content

fix(migrate): specify full version ranges for @next packages - #1292

Merged
AdrianGonz97 merged 5 commits into
version-1from
fix/specify-full-version-range
Aug 24, 2026
Merged

fix(migrate): specify full version ranges for @next packages#1292
AdrianGonz97 merged 5 commits into
version-1from
fix/specify-full-version-range

Conversation

@AdrianGonz97

@AdrianGonz97 AdrianGonz97 commented Aug 21, 2026

Copy link
Copy Markdown
Member

Closes #1283

One small quirk is if they have a package that has @sveltejs/kit@^2 as a peer-dependency (e.g. better-auth), then an install with npm will fail without the --legacy-peer-deps flag.

We could pass it automatically as part of npm install (which can work), however, all subsequent npm install's by the user will fail if they don't provide the flag themselves, which can be confusing if they're not educated about it. I wonder if we should log out some info telling them about it? ...but then we'd have to be able to detect if the install failed for that very reason, so we may be falling down an annoying rabbit hole just for that.

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@pkg-svelte-dev

pkg-svelte-dev Bot commented Aug 21, 2026

Copy link
Copy Markdown

Install the latest version of sv from e77af83:

pnx https://pkg.svelte.dev/sv/c/e77af83291594c860c8c24a88a1bcba8dacf43cd create

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1292

@changeset-bot

changeset-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e77af83

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

@svelte-docs-bot

Copy link
Copy Markdown

@AdrianGonz97
AdrianGonz97 marked this pull request as ready for review August 24, 2026 15:16

@jycouet jycouet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL ^x.y.z-next.0 is working!
Nice

@AdrianGonz97
AdrianGonz97 merged commit 8a9e893 into version-1 Aug 24, 2026
7 checks passed
@AdrianGonz97
AdrianGonz97 deleted the fix/specify-full-version-range branch August 24, 2026 16:03
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.

npm doesn't install next versions after migration

2 participants