-
Notifications
You must be signed in to change notification settings - Fork 1k
Add --dry-run flag to wrangler setup and also a dryRun option to runAutoConfig
#11258
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
Add --dry-run flag to wrangler setup and also a dryRun option to runAutoConfig
#11258
Conversation
🦋 Changeset detectedLatest commit: e6507ae The changes in this PR will be included in the next version bump. 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
@cloudflare/workers-utils
wrangler
commit: |
762dd4c to
f793929
Compare
penalosa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies, this is going to have horrific merge conflicts with #11251
yeah... 😅 🥲 |
…o `runAutoConfig`
Co-authored-by: Somhairle MacLeòid <[email protected]>
577a956 to
5b5a228
Compare
135aa5a to
4133078
Compare
4133078 to
40e06c0
Compare
Fixes https://jira.cfdata.org/browse/DEVX-2315
This PR adds a
--dry-runoption towrangler setupso that users can runwrangler setup --dry-runknowing that no local changes will actually be applied.It also adds a
dryRunoption to therunAutoConfigfunction that can be used to programmatically run the autoconfig flow in dry-run mode.wrangler setupand autoconfig are not in v3