-
Notifications
You must be signed in to change notification settings - Fork 10k
Update remote bindings to remove experimental #25064
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
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
@penalosa @dario-piotrowicz should |
|
Preview URL: https://adb392eb.preview.developers.cloudflare.com Files with changes (up to 15)
|
I am in favor for removing the |
| remoteProxySessionDetails = await experimental_maybeStartOrUpdateRemoteProxySession({ | ||
| bindings: { | ||
| MY_KV: { | ||
| type: 'kv_namespace', | ||
| id: 'kv-id', | ||
| remote: true, | ||
| } | ||
| } | ||
| }, | ||
| remoteProxySessionDetails | ||
| ); |
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.
I bet the broken indentation is not intentional 🙂
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.
should be fixed now 👍
Co-authored-by: Arunesh <[email protected]>
updated :) |
|
|
||
| - When you run a remote development session using the `--remote` flag, a limit of 50 [routes](/workers/configuration/routing/routes/) per zone is enforced. Learn more in[ Workers platform limits](/workers/platform/limits/#number-of-routes-per-zone-when-using-wrangler-dev---remote). | ||
|
|
||
| ``` |
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.
| ``` |
| remoteProxySessionDetails = await maybeStartOrUpdateRemoteProxySession({ | ||
| bindings: { | ||
| MY_KV: { | ||
| type: 'kv_namespace', | ||
| id: 'kv-id', | ||
| remote: true, | ||
| } | ||
| } | ||
| }, | ||
| remoteProxySessionDetails | ||
| ); |
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 needs formatted
Summary
With cloudflare/workers-sdk#10604, remote bindings are no longer experimental. This updates documentation of remote bindings to reflect that.