Skip to content

chore: svelte-kit-2 - #6103

Merged
alextran1502 merged 3 commits into
mainfrom
deps/svelte-kit-2
Jan 20, 2024
Merged

alextran1502 merged 3 commits into
mainfrom
deps/svelte-kit-2

Conversation

@jrasm91

@jrasm91 jrasm91 commented Jan 1, 2024 •

Copy link
Copy Markdown
Member

Upgrade to svelte kit 2 and vite 5.

  • Redirects no longer need to be thrown
  • Type imports are a lot more strict

@jrasm91
jrasm91 force-pushed the deps/svelte-kit-2 branch from 361f50e to c16a73c Compare January 1, 2024 19:38
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jan 1, 2024 •

Copy link
Copy Markdown

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: c959b16
Status: ✅  Deploy successful!
Preview URL: https://bb43cedd.immich.pages.dev
Branch Preview URL: https://deps-svelte-kit-2.immich.pages.dev

View logs

@jrasm91
jrasm91 requested a review from alextran1502 January 18, 2024 04:53
@jrasm91
jrasm91 marked this pull request as ready for review January 18, 2024 04:53

@danieldietzler danieldietzler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Can we have a linting rule that enforces type imports if we only use the import as a type and not as a value?

@alextran1502
alextran1502 merged commit 6e066aa into main Jan 20, 2024
@alextran1502
alextran1502 deleted the deps/svelte-kit-2 branch January 20, 2024 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants