-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: add app-state migration #354
Conversation
A simple, string-based migration for sveltejs/kit#13140 (tested on that repo, zero false-positives, less than five false negatives)
🦋 Changeset detectedLatest commit: 25a948b 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 |
commit: |
preview: https://svelte-dev-git-preview-cli-354-svelte.vercel.app/ this is an automated message |
Merged main here after merging #349 to have more sensible ordering of the migrations in the readme and website. Added the new migration to docs. Feel free to change the wording |
Should we make this an |
This already contains migrations for navigating (though the one with current on it) and updated - page-state is a misnomer, you're right that we should rename it to app-state |
A simple, string-based migration for sveltejs/kit#13140 (tested on that repo, zero false-positives, less than five false negatives)
We can switch to using the Svelte parser if people prefer that to get a bit better with regards to only actually transforming identifiers, but to me this seems to work fine.