Support/migrate to Svelte 5

[rChaoz]

Bump major and migrate library to Svelte 5

Closes #57

[rChaoz]

@sibiraj-s all the requested changes are done I think. I created a new PR to update the v1 docs that should be merged first if you want the v1 docs to have the correct installation command, but you'll have to update the tag also or create a new one

[sibiraj-s]

Thanks @rChaoz will review and try to get this merged today or tomorrow.

[alexbaramilis]

Any progress on this? I'm in the middle of migrating my codebase to Svelte 5 and this is my only blocker 🙏

[sibiraj-s]

Any progress on this? I'm in the middle of migrating my codebase to Svelte 5 and this is my only blocker 🙏

Will get it finished by today for sure. Sorry for the delay.

[sibiraj-s]

Thanks @rChaoz . 🎉 I did some cleanup on 88286d9, nothing major just stylistic mostly.

I removed $effect.pre call, I saw some comments those incomplete. Also, onMount accepts an async function now, when async cleanup callback doesn't work. which is not there anyway.

Let me know if I misunderstood that. I can fix it. Please see the commit 88286d9 and let me know if you have anythoughts.

Merging the existing changes now, since people are waiting on this.

[rChaoz]

Hi! I have just a question and a comment. First, why use

{#if children}
  {@render children()}
{/if children}

over

{@render children?.()}

? To my understanding they are the same.

Second, you should update the latest v1 tag to point to the merge made earlier (with the v1 docs), and the 2.0 README header was lost but I'll make a tiny PR for that.

[sibiraj-s]

Hi! I have just a question and a comment. First, why use

No reason, just preference, I like verbose when possible. both are absolutely same.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in the thread.