Added new pipelines commands (pipelines, streams, sinks, setup) [do not merge until 9/22]#10612
Conversation
🦋 Changeset detectedLatest commit: c89c187 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: |
penalosa
left a comment
There was a problem hiding this comment.
Just sinks—still need to review streams
oliy
left a comment
There was a problem hiding this comment.
lgtm from the perspective of New Pipeline API usage. Will have to defer to wrangler team for wrangler code/tool best practies.
d52a462 to
344beee
Compare
|
as discussed with team, please wait to land till monday (9/22) |
| }), | ||
| config.configPath, | ||
| args.env, | ||
| false // Don't offer to update automatically |
There was a problem hiding this comment.
This should be set to true, ideally. Why don't you want the config file updated automatically?
penalosa
left a comment
There was a problem hiding this comment.
Approved, but this is still blocked on merging
…d old pipelines commands behind --legacy
… points to stream id
da9e634 to
c89c187
Compare

Added new pipelines commands (pipelines, streams, sinks, setup), moved old pipelines commands behind --legacy
Approval: CR-1299934