Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
remove defaults and rely on API defaults instead
Browse files Browse the repository at this point in the history
cmackenzie1 committed Jan 24, 2025
1 parent c0b6533 commit d10d5c9
Showing 3 changed files with 5 additions and 7 deletions.
4 changes: 2 additions & 2 deletions packages/wrangler/src/__tests__/pipelines.test.ts
Original file line number Diff line number Diff line change
@@ -304,8 +304,8 @@ describe("pipelines", () => {
--r2-secret-access-key R2 service Secret Access Key for authentication. Leave empty for OAuth confirmation. [string]
--r2-prefix Prefix for storing files in the destination bucket [string] [default: \\"\\"]
--compression Compression format for output files [string] [choices: \\"none\\", \\"gzip\\", \\"deflate\\"] [default: \\"gzip\\"]
--file-template Template for individual file names (must include \${slug}) [string] [default: \\"\${slug}\${extension}\\"]
--partition-template Path template for partitioned files in the bucket [string] [default: \\"event_date=\${date}/hr=\${hr}\\"]
--file-template Template for individual file names (must include \${slug}) [string]
--partition-template Path template for partitioned files in the bucket. If not specified, the default will be used [string]
GLOBAL FLAGS
-c, --config Path to Wrangler configuration file [string]
7 changes: 3 additions & 4 deletions packages/wrangler/src/pipelines/cli/create.ts
Original file line number Diff line number Diff line change
@@ -157,16 +157,15 @@ export function addCreateOptions(yargs: Argv<CommonYargsOptions>) {
})
.option("partition-template", {
type: "string",
describe: "Path template for partitioned files in the bucket",
default: "event_date=${date}/hr=${hour}",
describe:
"Path template for partitioned files in the bucket. If not specified, the default will be used",
demandOption: false,
})
.option("file-template", {
type: "string",
describe: "Template for individual file names (must include ${slug})",
default: "${slug}${extension}",
demandOption: false,
coerce: (val: string) => {
coerce: (val) => {
if (!val.includes("${slug}")) {
throw new Error("filename must contain ${slug}");
}
1 change: 0 additions & 1 deletion packages/wrangler/src/pipelines/cli/update.ts
Original file line number Diff line number Diff line change
@@ -105,7 +105,6 @@ export function addUpdateOptions(yargs: Argv<CommonYargsOptions>) {
type: "string",
describe:
"PipelineTransform worker and entrypoint (<worker>.<entrypoint>)",
default: undefined,
demandOption: false,
})

0 comments on commit d10d5c9

Please sign in to comment.