Skip to content

Conversation

jycouet
Copy link
Contributor

@jycouet jycouet commented Sep 23, 2025

Closes: #721


Making the update script a bit more open to check templates & shared stuff.
Let me know if it's "too much" :)

Copy link

changeset-bot bot commented Sep 23, 2025

🦋 Changeset detected

Latest commit: c443857

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

Copy link

pkg-pr-new bot commented Sep 23, 2025

Open in StackBlitz

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

commit: c443857

@manuel3108
Copy link
Member

Same test issues as in #720 for whatever reason.
In the past we had some different opinions on this script overall. I don't think that those dicussions have been public though.

One of the main thoughts was that package managers are already fetching and installing the newest version for us if it's a first install (at least for patches, not sure about minors, don't think so for majors). So why should we bother keeping these up to date? But tbh, having a look at the div, this would barely make a difference in this case.

@jycouet
Copy link
Contributor Author

jycouet commented Sep 26, 2025

With the update tests are now good ✌️

@manuel3108 manuel3108 merged commit 55632de into main Sep 28, 2025
8 checks passed
@manuel3108 manuel3108 deleted the deps branch September 28, 2025 04:43
@github-actions github-actions bot mentioned this pull request Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants