-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chore: check WORKERS_CI env variable to determine if we're building for Cloudflare Workers
#13733
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
Changes from 29 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
0f0c26a
simplify workers and pages distinction
0004d36
support workers ci in adapter auto
fa08a55
add todo
1868b8c
remove todo
38b2940
improve error message
7bd425a
spacing
4a61e60
Update packages/adapter-auto/index.js
teemingc a9ce1c9
validate main key and fix relative import paths
9d2e796
Merge branch 'feat-adapter-auto-workers-ci' of https://github.com/sve…
24a6ea8
better comment
b60969b
allow spa static assets configuration
8d4169d
add validation error reasons
1417d6e
docs formatting
f6b2fcb
make compatibility date generic
515b1e6
mention workers ci
710510b
grammar
d37a58a
integrate #14871 thanks to penalosa
teemingc 0066267
Merge branch 'main' into feat-adapter-auto-workers-ci
teemingc 15e31af
add test for workers_ci
teemingc a3d3e80
format
teemingc daaf6bc
update test
teemingc 543f192
split out
teemingc 9bfdab7
split out more
teemingc 5028e2b
split
teemingc b622a38
add unit test
teemingc f64d9ea
Remove WORKERS_CI check from Cloudflare adapter
teemingc 7ccb7dc
Apply suggestion from @teemingc
teemingc e53f113
Apply suggestion from @teemingc
teemingc 44845f0
Merge branch 'version-3' into feat-adapter-auto-workers-ci
teemingc eb9cf06
Apply suggestion from @teemingc
teemingc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| '@sveltejs/adapter-cloudflare': patch | ||
| --- | ||
|
|
||
| chore: check the `WORKERS_CI` environment variable when determining if we're building for Cloudflare Workers | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.