Skip to content
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

Migrate to Svelte 5 #1866

Merged
merged 3 commits into from
Nov 1, 2024
Merged

Migrate to Svelte 5 #1866

merged 3 commits into from
Nov 1, 2024

Conversation

stolinski
Copy link
Collaborator

Ran the migration assistant. Site loads. Will be bug hunting now.

Copy link

vercel bot commented Oct 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
syntax-website ✅ Ready (Inspect) Visit Preview Oct 21, 2024 5:43pm

Copy link

codecov bot commented Oct 21, 2024

Bundle Report

Changes will decrease total bundle size by 525.66kB (-1.58%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
website-client-esm 2.14MB 162.1kB (-7.04%) ⬇️
website-server-esm 30.57MB 363.55kB (-1.18%) ⬇️

@stolinski
Copy link
Collaborator Author

All tests passing locally, svelte check all passing.

@stolinski
Copy link
Collaborator Author

For reviewers. It's a big pr, but since it was done using the migration assistant, doing smaller chunks was less possible. That said, all very smooth. I manually tested and ran playwrite / svelte check.

@stolinski
Copy link
Collaborator Author

Heyall. Speak now of forever hold your peace.

@stolinski stolinski merged commit 8c99918 into main Nov 1, 2024
4 of 5 checks passed
@stolinski stolinski deleted the svelte-5 branch November 1, 2024 17:06
github-actions bot added a commit that referenced this pull request Nov 1, 2024
* runs svelte 5 migration assistant

* fixes migration tasks

* 0 svelte check errors 2.1.166
Copy link

sentry-io bot commented Nov 1, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ DataCloneError: The object can not be cloned. /(site) View Issue
  • ‼️ TypeError: e(m).close is not a function. (In 'e(m).close()', 'e(m).close' is undefined) /(site) View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant