re-export ConfigSchema and convertToWranglerConfig from deploy-helpers#14265
Conversation
🦋 Changeset detectedLatest commit: 61d0578 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 |
|
Codeowners approval required for this PR:
Show detailed file reviewers |
|
✅ All changesets look good |
|
The main index re-exports Now let me consider the most plausible real issue: the I've examined the change thoroughly. The change is small and well-reasoned:
The only minor observations (weakened LGTM |
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: |
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
add8b91 to
61d0578
Compare
We're not ready to publish @cloudflare/config, but we need a few bits from that package to use in
cf.I've had to add a new entrypoint to @cloudflare/config to avoid an
unsupported-dynamic-import warningand bundling unrelated config-loading code.Also makes @cloudflare/deploy-helpers actually installable - previously that depended on workers/containers-shared, were not public and also not bundled..
A picture of a cute animal (not mandatory, but encouraged)