Repository navigation
[wrangler] Fix deploy warning snapshots - #16353
Conversation
|
|
Codeowners approval required for this PR:
Show detailed file reviewers |
|
LGTM |
|
✅ Approved — Test-only snapshot update with no issues found. The decision is advisory: this comment never approves, blocks or requests changes on the pull request. ✅ All checks passed📋 Code Quality: ✅ LGTMThe change updates the expected warning text in the deploy E2E snapshots. It is limited to Reviewed commit: |
@cloudflare/autoconfig
@cloudflare/build-output-utils
@cloudflare/codemods
@cloudflare/config
@cloudflare/containers-shared
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-functions
@cloudflare/pages-shared
@cloudflare/runtime-types
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-plugin
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
Fixes the deploy E2E snapshot failures blocking #16313 (failed job).
Update the
workers_devandpreview_urlswarning snapshots to namewrangler.toml, matching the fixture and the current deployment guidance. CI stops at the first failed assertion, so the adjacent preview URL snapshot needs the same update.Validation:
pnpm -w test:ci -F wrangler -- src/__tests__/deploy/workers-dev.test.ts— 57 tests passed.packages/wrangler/e2e/deploy.test.ts.No changeset is needed for this test-only change.
Note
This is a contribution from an AI agent: Codex (GPT-6), at a maintainer's request.