-
Notifications
You must be signed in to change notification settings - Fork 608
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
Warn that Hyperdrive bindings do not yet work in local dev #4144
Warn that Hyperdrive bindings do not yet work in local dev #4144
Conversation
🦋 Changeset detectedLatest commit: 69339ab The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/6502569521/npm-package-wrangler-4144 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/6502569521/npm-package-wrangler-4144 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/6502569521/npm-package-wrangler-4144 dev path/to/script.js Additional artifacts:npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/6502569521/npm-package-cloudflare-pages-shared-4144 Note that these links will no longer work once the GitHub Actions artifact expires.
| Please ensure constraints are pinned, and |
807726e
to
463febb
Compare
Codecov Report
@@ Coverage Diff @@
## main #4144 +/- ##
==========================================
+ Coverage 75.37% 75.41% +0.03%
==========================================
Files 223 223
Lines 12217 12217
Branches 3159 3159
==========================================
+ Hits 9209 9213 +4
+ Misses 3008 3004 -4 |
5556675
to
ba453a4
Compare
Support is already in progress (see cloudflare/workerd#1266) and should be coming in the next few weeks, but we've already had at least one user get very confused by this and cause us to spend time on debugging before realizing the problem was that they were talking about problems in wrangler dev rather than at the edge. This is an easy way to attempt to mitigate that for now.
ba453a4
to
69339ab
Compare
Fixed up yet another merge conflict (which is quite amazing on such a tiny change) |
Support is already in progress (see cloudflare/workerd#1266) and should be coming in the next few weeks, but we've already had at least one user get very confused by this and cause us to spend time on debugging before realizing the problem was that they were talking about problems in wrangler dev rather than at the edge. This is an easy way to attempt to mitigate that for now.
Fixes #4133
Author has included the following, where applicable:
Reviewer is to perform the following, as applicable: