-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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] add @sveltejs/adapter-static
to site package
#4245
[chore] add @sveltejs/adapter-static
to site package
#4245
Conversation
sveltejs#4011 added @sveltejs/adapter-static . But sveltejs#4239 removed it. Then deploy process is failed. (Please see sveltejs#4244 (comment) for more details)
|
sites/kit.svelte.dev/package.json
Outdated
@@ -9,6 +9,7 @@ | |||
}, | |||
"devDependencies": { | |||
"@sveltejs/adapter-auto": "workspace:*", | |||
"@sveltejs/adapter-static": "workspace:^1.0.0-next.29", |
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.
"@sveltejs/adapter-static": "workspace:^1.0.0-next.29", | |
"@sveltejs/adapter-static": "workspace:*", |
I think this should be workspace:*
like others are
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.
This is simply rollback to previous commit.
https://github.com/sveltejs/kit/blame/cdc32a9f4f17d69be6ca1cae58bf3961155545a2/sites/kit.svelte.dev/package.json
But workspace:*
is better I think too.
pnpm-lock.yaml
Outdated
@@ -391,6 +391,7 @@ importers: | |||
sites/kit.svelte.dev: | |||
specifiers: | |||
'@sveltejs/adapter-auto': workspace:* | |||
'@sveltejs/adapter-static': workspace:^1.0.0-next.29 |
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.
'@sveltejs/adapter-static': workspace:^1.0.0-next.29 | |
'@sveltejs/adapter-static': workspace:* |
I think this should be workspace:*
like others are
thanks! |
Now deploy process seems failed. (e.g. https://app.netlify.com/sites/kit-demo/deploys/622468f183e8a9000860529a#L1048)
This PR fixes this issue.
Please see the below comments for more details.
#4244 (comment)
#4244 (comment)
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm test
and lint the project withpnpm lint
andpnpm check
Changesets
pnpx changeset
and following the prompts. All changesets should bepatch
until SvelteKit 1.0