-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Migration script #5773
Migration script #5773
Conversation
|
This is a bit loosey-goosey (I didn't add tests, on the basis that this is somewhat throwaway, but I could be persuaded of their value) but probably far enough along that we can start trying it out on codebases to see how far we get. |
…support server arrow function migration, extract migration into own file and add tests
alright, time to give this a whirl |
This adds a new package,
svelte-migrate
, which will automate the automate-able bits of #5748:It doesn't attempt to boil the ocean — it just moves files to their new homes and annotates the code with manual TODOs, which will be documented in a GitHub discussion.
(For clarity, we're doing the migration script first so we can dogfood it on our own apps and test suites, but we haven't implemented the changes yet, so don't get ahead of yourselves and run this on your own apps just yet!)
Bunch of stuff that still needs to happen:
+page.server.js
and+server.js
telling people to update handler return values+page.js
and+layout.js
telling people to updateload
return valuesdata
prop+error
can't have load, somehow./__types/whatever
with./$types