-
Notifications
You must be signed in to change notification settings - Fork 239
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
Svelte 5 support #186
Comments
Thanks a lot for your work ! It's a great way to have a preview of Svelte 5.
|
I would be happy to include your work into the main branch. And to share CP to show directly Svelte 4 VS Svelte 5, we can share this url: https://component-party.dev/?f=svelte,svelte5 What do you think ? |
Yeah, I've been going back and forth on this. I'd originally had it as
Hmm, I'm not sure about this one. For now I'll use the old method and we can take the discussion to #183 and discuss it separately.
Sounds great! |
I put together a branch for Svelte 5 support: https://github.com/benmccann/component-party.dev/tree/demo
Is this something you'd want me to PR now or wait until it's released? It's possible the syntax will change before release - though I imagine it shouldn't be too hard to update if a rune gets renamed or something like that and I'd be happy to send a PR for it.
You can check it out here: https://component-party-runes.vercel.app/
My branch shows Svelte 4 vs Svelte 5 by default just for the deployed demo, but I could revert that part of the change. The only other thing to mention is that I disabled linting against the Svelte 5 version until
eslint-plugin-svelte
adds support for runes.The text was updated successfully, but these errors were encountered: