feat(wrangler): add binding overrides to test harness#14446
Conversation
🦋 Changeset detectedLatest commit: 08c60ff 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 |
|
Codeowners approval required for this PR:
Show detailed file reviewers |
|
Both The PR is well-implemented. The core logic correctly merges binding overrides via I reviewed PR #14446 thoroughly: the binding override mechanism ( LGTM |
|
✅ All changesets look good |
@cloudflare/autoconfig
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-pool-workers
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
4bc0083 to
ef732c6
Compare
90311dd to
ede3459
Compare
petebacondarwin
left a comment
There was a problem hiding this comment.
A couple of nits but this looks good to me.
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
Fixes n/a.
A picture of a cute animal (not mandatory, but encouraged)