-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
chore: migrate config.kit.files.hooks #251
Conversation
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
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.
👍
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.
Thank you!
I didn't get the reason but CI failed... |
Something seems to go wrong when using Error: Error: 404 /_app/immutable/assets/logo-06fafbe9.svg does not begin with `base`, which is configured in `paths.base` and can be imported from `$app/paths` (linked from /eslint-plugin-svelte/) HTML generated by the current job: <img src="/_app/immutable/assets/logo-06fafbe9.svg" alt="Logo" class="svelte-jet77t"> Current published HTML: <img src="/eslint-plugin-svelte/_app/immutable/assets/logo-06fafbe9.svg" alt="Logo" class="svelte-jet77t"> |
I found the cause. export function assets_base(config) {
return config.paths.assets + '/' || config.paths.base + '/' || './';
} The first expression will be 100% truthy. I'll check if it's already been reported. |
I opened an issue. |
I opened a PR. |
Update
svelte.config.mjs
according to sveltejs/kit#6586