Wrangler preview urls e2e fixes and warning follow up#10561
Conversation
🦋 Changeset detectedLatest commit: 96768ea The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
create-cloudflare
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-editor-shared
wrangler
commit: |
That sounds bad! What the initial PR merged with failing E2E? |
Seems like it was somehow skipped? Just fixed some snapshot lines. |
3d6d9c2 to
58bc92b
Compare
58bc92b to
8c2bbfa
Compare
8c2bbfa to
d007e1e
Compare
petebacondarwin
left a comment
There was a problem hiding this comment.
You ticked that this PR doesn't need a backport because it is "not a new feature". But patches to current features are exactly what does need backporting. The only reason this might not be backported is if the feature being patched is not in Wrangler v3. Is that the case?
6d5f22a to
c590603
Compare
…g to false Fixes existing tests, and adds a new test for `preview_urls = true`.
…st deploy These warnings are just noise on a first deploy, not really necessary.
3bc4b62 to
96768ea
Compare
Follow up of: #10478
Describe your change...