-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Conversation
…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.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
There was a problem hiding this 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"], |
There was a problem hiding this comment.
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
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.