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
Implement use of .svelte-kit with hot-reloading #540
Conversation
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.
Looks good!!
0ee88e1
to
2496426
Compare
2496426
to
fc65683
Compare
Had a hard time working with webpack and its ts types, ended up migrating the config-file away from js, but keeping the old one since it's used by build:client-old. |
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.
What happened to the other webpack configs? Shouldn't this be a rename of the old file?
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.
#540 (comment)
^^^^^^^
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.
ah sorry 🙃
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.
LGTM
Some svelte-types are only inferred correctly when tsconfig extends .svelte-kit/tsconfig. This has therefore been an attempt to combine the two files.
The setup to use
<script lang="ts">
in svelte was also not fully configured, and required a ts-bump to be compatible with our project.The bump has also led to refactoring the code to new ts-syntax.