Skip to content

fix(examples): with-svelte example TypeScript #10450

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

Merged
merged 3 commits into from
May 6, 2025
Merged

fix(examples): with-svelte example TypeScript #10450

merged 3 commits into from
May 6, 2025

Conversation

anthonyshew
Copy link
Contributor

@anthonyshew anthonyshew commented May 6, 2025

Description

Inspired by @DoctorRyner in #10437, a set of changes to make TypeScript work better for the with-svelte example.

Testing Instructions

This set of changes works correctly in my editor.

DoctorRyner and others added 3 commits May 3, 2025 18:15
…rrors

I tested it with VS Code and Helix Editor and with current config, all export/imports of files, for instance in index.ts config, will result in a false error reporting of "Module does not exist", it seems to be happening because of miss configuration.
…oject

Previously tsconfig.json refered to a file that does not exist.
@anthonyshew anthonyshew requested a review from a team as a code owner May 6, 2025 20:46
@turbo-orchestrator turbo-orchestrator bot added the area: examples Improvements or additions to examples label May 6, 2025
Copy link

vercel bot commented May 6, 2025

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

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-gatsby-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-kitchensink-blog ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-svelte-web ❌ Failed (Inspect) May 6, 2025 8:50pm
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
examples-vite-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 8:50pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
turbo-site ⬜️ Skipped (Inspect) May 6, 2025 8:50pm

Copy link
Member

@chris-olszewski chris-olszewski left a comment

Choose a reason for hiding this comment

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

Approving, but this still can result in failures if checking types before building

@@ -1,15 +1,10 @@
{
"extends": "@repo/typescript-config/svelte.json",
"extends": ["@repo/typescript-config/svelte.json", "./.svelte-kit/tsconfig.json"],
Copy link
Member

Choose a reason for hiding this comment

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

.svelte-kit only gets generated after a build is run so running check-types before building will result in failure

@anthonyshew anthonyshew linked an issue May 6, 2025 that may be closed by this pull request
1 task
@anthonyshew anthonyshew changed the title fix(examples): TypeScript in with-svelte example fix(examples): with-svelte example TypeScript May 6, 2025
@anthonyshew anthonyshew merged commit 3391ae4 into main May 6, 2025
31 of 35 checks passed
@anthonyshew anthonyshew deleted the shew/97a20 branch May 6, 2025 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: examples Improvements or additions to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

with-svelte template: Incorrect tsconfig.json file for @repo/ui package
3 participants